Skip to content

Commit a663119

Browse files
authored
[LV] Fix verifier failures due to 93073af (#162097)
Follow up on 93073af ([LV] Move 3 functions into VPlanTransforms (NFC)) to not call runPass on the moved functions, as that results in verifier failures. Ref: https://lab.llvm.org/buildbot/#/builders/187/builds/12178
1 parent 35c57a7 commit a663119

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

llvm/lib/Transforms/Vectorize/LoopVectorize.cpp

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -8393,11 +8393,11 @@ VPlanPtr LoopVectorizationPlanner::tryToBuildVPlanWithVPRecipes(
83938393
R->setOperand(1, WideIV->getStepValue());
83948394
}
83958395

8396-
VPlanTransforms::runPass(
8397-
VPlanTransforms::addExitUsersForFirstOrderRecurrences, *Plan, Range);
8396+
// TODO: We can't call runPass on these transforms yet, due to verifier
8397+
// failures.
8398+
VPlanTransforms::addExitUsersForFirstOrderRecurrences(*Plan, Range);
83988399
DenseMap<VPValue *, VPValue *> IVEndValues;
8399-
VPlanTransforms::runPass(VPlanTransforms::addScalarResumePhis, *Plan,
8400-
RecipeBuilder, IVEndValues);
8400+
VPlanTransforms::addScalarResumePhis(*Plan, RecipeBuilder, IVEndValues);
84018401

84028402
// ---------------------------------------------------------------------------
84038403
// Transform initial VPlan: Apply previously taken decisions, in order, to
@@ -8508,8 +8508,9 @@ VPlanPtr LoopVectorizationPlanner::tryToBuildVPlan(VFRange &Range) {
85088508
DenseMap<VPValue *, VPValue *> IVEndValues;
85098509
// TODO: IVEndValues are not used yet in the native path, to optimize exit
85108510
// values.
8511-
VPlanTransforms::runPass(VPlanTransforms::addScalarResumePhis, *Plan,
8512-
RecipeBuilder, IVEndValues);
8511+
// TODO: We can't call runPass on the transform yet, due to verifier
8512+
// failures.
8513+
VPlanTransforms::addScalarResumePhis(*Plan, RecipeBuilder, IVEndValues);
85138514

85148515
assert(verifyVPlanIsValid(*Plan) && "VPlan is invalid");
85158516
return Plan;

0 commit comments

Comments
 (0)