[llvm] [MachinePipeliner] Limit the number of stores in BB (PR #154940)
via llvm-commits
llvm-commits at lists.llvm.org
Fri Aug 22 10:23:58 PDT 2025
aankit-ca 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 [hexagon compiler runs "forever" on matrix-spec-types at O2 #150262](https://github.com/llvm/llvm-project/issues/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!
Thanks for looking into this issue @kasuga-fj . I'm on a vacation right now and will be back on Sep 2. I'll verify this once I'm back!
https://github.com/llvm/llvm-project/pull/154940
More information about the llvm-commits
mailing list