reactorlabs / rir

GNU General Public License v2.0
62 stars 18 forks source link

Kill time based heuristics no versions #1285

Closed fikovnik closed 1 month ago

fikovnik commented 1 month ago
  1. Remove the type feedback versioning. The rationale is that it likely does not add much value while being costly. We used the information about version update before to give extra chance to recompile, but that might be either unproductive or unprecise. Unproductive in the sense that maybe the new information is not useful for the given context. Unprecise in the sense that it does not track the possible type feedback version change in inlinees.

  2. Removes the time base heuristics for recompilation.

  3. Fixes some issues related to inlining

o- commented 1 month ago

Here are some stats for your PR:

Please find your performance results at https://rir-benchmarks.prl.fit.cvut.cz/diff?job_ids[]=7025932406&job_ids[]=6997286723&selection=all