!fixup add TODO for negative steps/reverse accesses

fhahn · fhahn · commit c6ce3094e72a · 2025-11-26T15:37:20.000Z
diff --git a/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp b/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
@@ -4065,6 +4065,7 @@ static bool isConsecutiveLoad(VPValue *Addr, Type *LoadTy, ScalarEvolution &SE,
 
   TypeSize TS = DL.getTypeStoreSize(LoadTy);
   const SCEV *ElementSizeSCEV = SE.getSizeOfExpr(StepSCEV->getType(), TS);
+  // TODO: Extend support to reverse accesses (with negative steps).
   return SE.isKnownPositive(StepSCEV) && StepSCEV == ElementSizeSCEV;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -4065,6 +4065,7 @@ static bool isConsecutiveLoad(VPValue Addr, Type LoadTy, ScalarEvolution &SE,`
`4065`	`4065`
`4066`	`4066`	`TypeSize TS = DL.getTypeStoreSize(LoadTy);`
`4067`	`4067`	`const SCEV *ElementSizeSCEV = SE.getSizeOfExpr(StepSCEV->getType(), TS);`
	`4068`	`+ // TODO: Extend support to reverse accesses (with negative steps).`
`4068`	`4069`	`return SE.isKnownPositive(StepSCEV) && StepSCEV == ElementSizeSCEV;`
`4069`	`4070`	`}`
`4070`	`4071`