[llvm] [ARM][ConstantIslands] Correct MinNoSplitDisp calculation (PR #114590)
via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 13 17:52:42 PST 2024
pzhengqc wrote:
> It has been a while since I looked into the constant island pass, but this sounds OK. As you mention this should only kick in when we are already 15 iterations in, so is aiming to converge quickly at the expense of optimality. Did you run any checks (the llvm-test-suite or a boot strap), that can check we don't iterate for a long time now? Thanks.
Yes, @davemgreen, I've run some checks (the llvm-test-suite, a boot strap, spec2000 and spec2006), but none of these tests seems to take more than 15 iterations for the constant island pass to converge. I guess, in reality, it's very rare for the check `if (CloserWater && MinNoSplitDisp > U.getMaxDisp() / 2)` to be true.
https://github.com/llvm/llvm-project/pull/114590
More information about the llvm-commits
mailing list