[llvm] [AMDGPU][MISched] Allow memory ops of different base pointers to be clustered (PR #140674)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Thu May 22 03:31:32 PDT 2025


================
@@ -585,10 +585,24 @@ bool SIInstrInfo::shouldClusterMemOps(ArrayRef<const MachineOperand *> BaseOps1,
     const MachineInstr &FirstLdSt = *BaseOps1.front()->getParent();
     const MachineInstr &SecondLdSt = *BaseOps2.front()->getParent();
 
-    if (!DisableDiffBasePtrMemClustering) {
+    if (EnableDiffBasePtrMemClustering) {
       // Only consider memory ops from same addrspace for clustering
       if (!memOpsHaveSameAddrspace(FirstLdSt, BaseOps1, SecondLdSt, BaseOps2))
         return false;
+
+      // Don't cluster scalar and vecter memory ops
----------------
arsenm wrote:

```suggestion
      // Don't cluster scalar and vector memory ops
```

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


More information about the llvm-commits mailing list