[PATCH] D147892: [VPlan] Unify Value2VPValue and VPExternalDefs maps (NFCI).
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sat Apr 15 08:36:06 PDT 2023
fhahn updated this revision to Diff 513915.
fhahn marked an inline comment as done.
fhahn added a comment.
In D147892#4265819 <https://reviews.llvm.org/D147892#4265819>, @Ayal wrote:
> This indeed deserves cleaning up!
> Would it be correct and reasonable to rename "External[Def]" >> "LiveIn[VPValue]" throughout, as in !hasDefiningRecipe()?
Update, thanks!
> Value2VPValueEnabled presumably applies to recipe'd VPValues only, i.e., excluding live-in VPValues? Better decrease allowed overriding cases, rather than increase them.
I updated the exsting asserts to allow querying and adding live-ins regardless of Value2VPValueEnabled and consolidated the functions.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D147892/new/
https://reviews.llvm.org/D147892
Files:
llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
llvm/lib/Transforms/Vectorize/VPlan.cpp
llvm/lib/Transforms/Vectorize/VPlan.h
llvm/lib/Transforms/Vectorize/VPlanHCFGBuilder.cpp
llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
llvm/unittests/Transforms/Vectorize/VPlanHCFGTest.cpp
llvm/unittests/Transforms/Vectorize/VPlanTest.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D147892.513915.patch
Type: text/x-patch
Size: 14645 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230415/c2cc9fc2/attachment.bin>
More information about the llvm-commits
mailing list