[llvm] [VPlan] Fix nowrap flags for strided access pointers from SCEV (PR #209453)
Florian Hahn via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 15 02:26:53 PDT 2026
================
@@ -7787,16 +7787,16 @@ void VPlanTransforms::convertToStridedAccesses(VPlan &Plan,
auto *AddRecPtr = cast<SCEVAddRecExpr>(PtrSCEV);
auto *Offset = Builder.createOverflowingOp(
Instruction::Mul, {CanIV, StrideInBytes},
- {AddRecPtr->hasNoUnsignedWrap(), AddRecPtr->hasNoSignedWrap()});
- auto *BasePtr = Builder.createNoWrapPtrAdd(
- StartVPV, Offset,
- AddRecPtr->hasNoUnsignedWrap() ? GEPNoWrapFlags::noUnsignedWrap()
- : GEPNoWrapFlags::none());
+ {AddRecPtr->hasNoUnsignedWrap(), /*HasNSW=*/false});
+ GEPNoWrapFlags NWFlags = AddRecPtr->hasNoUnsignedWrap()
+ ? GEPNoWrapFlags::noUnsignedWrap()
+ : GEPNoWrapFlags::none();
+ auto *BasePtr = Builder.createNoWrapPtrAdd(StartVPV, Offset, NWFlags);
----------------
fhahn wrote:
nit: while tocuhing the line, could make consistent with most other code in the function
```suggestion
VPValue *BasePtr = Builder.createNoWrapPtrAdd(StartVPV, Offset, NWFlags);
```
https://github.com/llvm/llvm-project/pull/209453
More information about the llvm-commits
mailing list