This patch adds a new prepareForExecution helper to set up live-ins, so
VPTransformState doesn't need to hold values like TripCount.
This also requires making the trip count operand for ActiveLaneMask
explicit in VPlan.
Paths
| Differential D116320
[VPlan] Add prepareForExecution to set up live-ins (NFC). ClosedPublic Authored by fhahn on Dec 27 2021, 1:05 PM.
Details Summary This patch adds a new prepareForExecution helper to set up live-ins, so This also requires making the trip count operand for ActiveLaneMask
Diff Detail
Unit TestsFailed
Event TimelineHerald added subscribers: tschuett, psnobl, rogfer01 and 2 others. · View Herald TranscriptDec 27 2021, 1:05 PM Comment Actions This looks good to me, adding some optional suggestions.
This revision is now accepted and ready to land.Dec 28 2021, 1:31 AM This revision was landed with ongoing or failed builds.Dec 28 2021, 8:51 AM Closed by commit rG9d297c7894ec: [VPlan] Add prepareToExecute to set up live-ins (NFC). (authored by fhahn). · Explain Why This revision was automatically updated to reflect the committed changes. fhahn marked 6 inline comments as done.
Revision Contents
Diff 396330 llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
llvm/lib/Transforms/Vectorize/VPlan.h
llvm/lib/Transforms/Vectorize/VPlan.cpp
|
Induction/CanonicalIV is another live-in that could be handled by prepareForExecution(), but soon D113223 will replace it and pass CanonicalIVStartValue to prepareForExecution() instead?