[llvm] [MachinePipeliner] Limit the number of stores in BB (PR #154940)
Ryotaro Kasuga via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 22 06:24:11 PDT 2025
kasuga-fj wrote:
TODO: Add some comments and tests
@aankit-ca Could you please run your benchmark with this change? I believe this is the simplest solution for #150262. If you're okay with it, I'd like to proceed with this approach. Here are some notes:
- I have confirmed that this patch resolves the specific case raised in #150262, but this change may reduce optimization opportunities for other cases.
- The default value of `SwpMaxNumStores` is arbitrary. Please let me know if you have any preferences.
- If we're very lucky, this might also improve compilation time in other cases, without causing any performance regressions.
Thanks in advance!
https://github.com/llvm/llvm-project/pull/154940
More information about the llvm-commits
mailing list