[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 07:53:24 PDT 2024


asb wrote:

> #73789 added load clustering and #73796 tried to add store clustering. If post machine schedule is used, previous cluster of load/store which formed in machine schedule may break. In order to solve this, add load/sotre clustering to post machine schedule.

Hi, I'm struggling to parse this a bit. I think you're saying that if load and store clustering aren't added to the post machine schedule in addition to being in `createMachineScheduler`, you see issues. Is that correct? It looks like PPC (which does store clustering only) also adds it in both functions, though AArch64 doesn't.

If you could update this PR to show any test differences that would really help with review. Thanks!

https://github.com/llvm/llvm-project/pull/111504


More information about the llvm-commits mailing list