[PATCH] D95383: [LV] Remove VPCallback.

[Gil Rapaport via Phabricator via llvm-commits]

gilr added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:4393
+      if (isa<Constant>(ScIncV) ||
+          OrigLoop->contains(cast<Instruction>(ScIncV)))
+        State.Plan->getOrAddVPValue(ScIncV);
----------------
Should this be `!OrigLoop->contains()` to match invariants? (why check at all?)


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:9109
 
 void VPTransformState::set(VPValue *Def, Value *IRDef, Value *V,
                            const VPIteration &Instance) {
----------------
Aren't the IRDef parameter(s) now redundant?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95383