[PATCH] D100258: [VPlan] Add first VPlan version of sinkScalarOperands.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp:128
+              auto *UI = dyn_cast<VPRecipeBase>(U);
+              return UI && UI->getParent() != VPBB;
+            }))
----------------
gilr wrote:
> Should be `return !UI || UI->getParent() != VPBB`?
Yes I think so, great catch!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100258