Skip to content

Commit 708f6fe

Browse files
committed
Return true from planContainsAdditionalSimplifications
1 parent 70b1b7f commit 708f6fe

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

llvm/lib/Transforms/Vectorize/LoopVectorize.cpp

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7606,11 +7606,9 @@ static bool planContainsAdditionalSimplifications(VPlan &Plan,
76067606
}
76077607
continue;
76087608
}
7609-
// A partial reduction is very differently costed compared to its
7610-
// underlying add instruction that the legacy cost model sees, so consider
7611-
// it an additional simplification.
7609+
// The VPlan-based cost model is more accurate for partial reduction and comparing against the legacy cost isn't desirable.
76127610
if (dyn_cast<VPPartialReductionRecipe>(&R))
7613-
continue;
7611+
return true;
76147612
if (Instruction *UI = GetInstructionForCost(&R))
76157613
SeenInstrs.insert(UI);
76167614
}

0 commit comments

Comments
 (0)