[llvm] [ARM] Speedups for CombineBaseUpdate. (PR #129725)
Peter Smith via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 5 03:31:53 PST 2025
https://github.com/smithp35 commented:
For a non-expert this looks reasonable to me. I've made a couple of stylistic comments but nothing significant.
I guess the biggest decision point is whether 64 is the right choice. It seems to be the right choice for the problematic test case, and sounds plausible to be enough for unrolled loops.
I'm inclined to say let's give it a try and tune it there's any evidence of regressions.
https://github.com/llvm/llvm-project/pull/129725
More information about the llvm-commits
mailing list