We should revisit the recheck == 97 condition. It does not play well with the current recompilation heuristic which decides to recompile every PIR_WARMUP invocations to the function (currently 100), so both conditions must be true simultaneously. Also, recheck is a global variable and other calls most likely increase it in between.
In other words, the chances it will trigger are low.
|
if (fail || (++recheck == 97 && RecompileHeuristic(fun))) { |
We should revisit the
recheck == 97condition. It does not play well with the current recompilation heuristic which decides to recompile every PIR_WARMUP invocations to the function (currently 100), so both conditions must be true simultaneously. Also,recheckis a global variable and other calls most likely increase it in between.In other words, the chances it will trigger are low.
rir/rir/src/compiler/native/builtins.cpp
Line 1416 in 95582fb