[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