[PATCH] D157194: [VPlan] Model wrap flags directly, remove *NUW opcodes (NFC)

[Ayal Zaks via Phabricator via llvm-commits]

Ayal added a comment.

A couple of post-commit nits.



================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.h:972
+    assert(OpType == OperationType::OverflowingBinOp &&
+           "recipe doesn't have a NUW flag");
+    return WrapFlags.HasNSW;
----------------



================
Comment at: llvm/lib/Transforms/Vectorize/VPlanRecipes.cpp:323
     Value *Step = createStepForVF(Builder, IV->getType(), State.VF, Part);
-    return Builder.CreateAdd(IV, Step, Name, IsNUW, false);
+    return Builder.CreateAdd(IV, Step, Name, hasNoUnsignedWrap(), false);
   }
----------------



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D157194/new/

https://reviews.llvm.org/D157194