[llvm] [RISCV] Add load/store clustering in post machine schedule (PR #111504)
Alex Bradbury via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 10 09:23:24 PDT 2024
asb wrote:
> > > > createGenericSchedLive
> > >
> > >
> > > I think the reason is you are using pre-ra scheduler here.
> >
> >
> > Thank you - the problem was indeed me being too eager with copy and paste and failing to switch to `createGenericSchedPostRA`. Now I've done that though, I see zero codegen changes within the in-tree unit tests.
>
> Thanks! I will ask @BoyaoWang430 to add some MIR tests. And thanks in advance if you can help to evaluate/review this PR. :-)
Thanks! And more generally, any note you have on how/if it affects codegen on external codebases very welcome. e.g. is this something that kicks in a lot in real-world code but we just don't trigger in our tests, or is it fairly rare it makes a difference (but of course worth addressing for the cases it helps)
https://github.com/llvm/llvm-project/pull/111504
More information about the llvm-commits
mailing list