[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