[llvm] [VPlan] Use SCEV to prove non-aliasing for stores at different offsets. (PR #170347)
Ramkumar Ramachandra via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 8 06:00:38 PST 2025
================
@@ -139,14 +140,77 @@ bool VPlanTransforms::tryToConvertVPInstructionsToVPRecipes(
return true;
}
-// Check if a memory operation doesn't alias with memory operations in blocks
-// between FirstBB and LastBB using scoped noalias metadata.
-// For load hoisting, we only check writes in one direction.
-// For store sinking, we check both reads and writes bidirectionally.
+/// Helper for extra no-alias checks via known-safe recipe and SCEV.
+class SinkStoreInfo {
+ const SmallPtrSetImpl<VPRecipeBase *> &ExcludeRecipes;
+ VPReplicateRecipe &GroupLeader;
+ ScalarEvolution &SE;
+ const Loop &L;
+ VPTypeAnalysis &TypeInfo;
+
+ // Return true if \p A and \p B are known to not alias for all VFs in the
+ // plan, checked via the distance between the accesses
+ bool isNoAliasViaDistance(VPReplicateRecipe *A, VPReplicateRecipe *B) const {
+ if (A->getOpcode() != Instruction::Store ||
+ B->getOpcode() != Instruction::Store)
+ return false;
+
+ VPValue *AddrA = A->getOperand(1);
+ const SCEV *SCEVA = vputils::getSCEVExprForVPValue(AddrA, SE, &L);
+ VPValue *AddrB = B->getOperand(1);
+ const SCEV *SCEVB = vputils::getSCEVExprForVPValue(AddrB, SE, &L);
+ if (isa<SCEVCouldNotCompute>(SCEVA) || isa<SCEVCouldNotCompute>(SCEVB))
+ return false;
+
+ const APInt *Distance;
+ if (!match(SE.getMinusSCEV(SCEVA, SCEVB), m_scev_APInt(Distance)))
+ return false;
+
+ const DataLayout &DL = L.getHeader()->getModule()->getDataLayout();
----------------
artagnon wrote:
```suggestion
const DataLayout &DL = SE.getDataLayout();
```
https://github.com/llvm/llvm-project/pull/170347
More information about the llvm-commits
mailing list