[llvm] [llvm][ConstraintElimination]Insert ConditionFact into loop header in case of monotonic induction variables (PR #112080)
Yingwei Zheng via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 21 21:02:19 PDT 2024
dtcxzyw wrote:
> I see that the compilation time "by files" is improved, but "by projects" is worsened.
It is possible since I calculate them with geomean.
> Is it acceptable? If not, what should be the threshold?
It is acceptable for me. Overall compile time regression < 0.3% should be OK since this patch eliminates many branches.
The IR diff looks OK. Wait for @fhahn and @nikic to see if there are correctness issues :)
I am sorry I have no bandwidth to take a deep look.
https://github.com/llvm/llvm-project/pull/112080
More information about the llvm-commits
mailing list