Skip to content

Commit

Permalink
[VPlan] Remove uneeded needsVectorIV check.
Browse files Browse the repository at this point in the history
Remove one of the last remaining uses of ::needsVectorIV, preparing for
its removal. Now that usesScalars is available and based on the
information explicit in VPlan, there is no need to use the pre-computed
needsVectorIV.

Reviewed By: Ayal

Differential Revision: https://reviews.llvm.org/D123720
  • Loading branch information
fhahn committed Apr 28, 2022
1 parent 50d648b commit 43842b8
Showing 1 changed file with 1 addition and 9 deletions.
10 changes: 1 addition & 9 deletions llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
Expand Up @@ -429,15 +429,7 @@ void VPlanTransforms::optimizeInductions(VPlan &Plan, ScalarEvolution &SE) {
IV->getStartValue(), Step, TruncI ? TruncI->getType() : nullptr);
HeaderVPBB->insert(Steps, HeaderVPBB->getFirstNonPhi());

// If there are no vector users of IV, simply update all users to use Step
// instead.
if (!IV->needsVectorIV()) {
IV->replaceAllUsesWith(Steps);
continue;
}

// Otherwise only update scalar users of IV to use Step instead. Use
// SetVector to ensure the list of users doesn't contain duplicates.
// Update scalar users of IV to use Step instead. Use SetVector to ensure the list of users doesn't contain duplicates.
SetVector<VPUser *> Users(IV->user_begin(), IV->user_end());
for (VPUser *U : Users) {
VPRecipeBase *R = cast<VPRecipeBase>(U);
Expand Down

0 comments on commit 43842b8

Please sign in to comment.