[llvm] MachineScheduler: Improve instruction clustering (PR #137784)

via llvm-commits llvm-commits at lists.llvm.org
Thu May 29 05:50:53 PDT 2025


ruiling wrote:

> Even after this patch, we still add weak "cluster" edges to the DAG. Is that still necessary?
> 
> Oh I see you mentioned that already. The DAG edges are only needed for some sort of check in MacroFusion, right? I don't really understand that part.

For the edge in `clusterNeighboringMemOps`, Removing it just causes more test changes. That's one of the reasons why I leave it there. I slightly prefer we remove it but no clear idea yet. I think it needs some more investigation, and it would be better done separately.

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


More information about the llvm-commits mailing list