Skip to content
Snippets Groups Projects
Commit 16640a59 authored by Matthew Gaudet's avatar Matthew Gaudet
Browse files

Bug 1813624 - Remove old IonBuilder warmup heuristic r=jandem

This was added in this [1] commit, but the actual purpose and intent have been
lost, though the comment has nevertheless survived.

[1]: https://hg.mozilla.org/mozilla-central/rev/251540232d8f8890458446fcb2e89f8a87b02cd7

Differential Revision: https://phabricator.services.mozilla.com/D168288
parent 8a803fd8
No related branches found
No related tags found
No related merge requests found
......@@ -13304,10 +13304,6 @@ bool CodeGenerator::link(JSContext* cx, const WarpSnapshot* snapshot) {
return false;
}
 
// Check to make sure we didn't have a mid-build invalidation. If so, we
// will trickle to jit::Compile() and return Method_Skipped.
uint32_t warmUpCount = script->getWarmUpCount();
IonCompilationId compilationId =
cx->runtime()->jitRuntime()->nextCompilationId();
JitZone* jitZone = cx->zone()->jitZone();
......@@ -13329,14 +13325,6 @@ bool CodeGenerator::link(JSContext* cx, const WarpSnapshot* snapshot) {
return true;
}
 
// IonMonkey could have inferred better type information during
// compilation. Since adding the new information to the actual type
// information can reset the usecount, increase it back to what it was
// before.
if (warmUpCount > script->getWarmUpCount()) {
script->incWarmUpCounter(warmUpCount - script->getWarmUpCount());
}
uint32_t argumentSlots = (gen->outerInfo().nargs() + 1) * sizeof(Value);
 
// We encode safepoints after the OSI-point offsets have been determined.
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment