[PATCH] D100258: [VPlan] Add first VPlan version of sinkScalarOperands.
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed May 5 14:04:05 PDT 2021
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
More information about the llvm-commits
mailing list