[llvm] [LoopVectorize] In LoopVectorize.cpp start using getSymbolicMaxBackedgeTakenCount (PR #108833)

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 1 13:02:37 PDT 2024


================
@@ -9584,7 +9584,7 @@ static bool processLoopInVPlanNativePath(
     ProfileSummaryInfo *PSI, LoopVectorizeHints &Hints,
     LoopVectorizationRequirements &Requirements) {
 
-  if (isa<SCEVCouldNotCompute>(PSE.getBackedgeTakenCount())) {
+  if (isa<SCEVCouldNotCompute>(PSE.getSymbolicMaxBackedgeTakenCount())) {
----------------
fhahn wrote:

>  can add a negative test for the native vplan case that proves we don't vectorise outer loops with an early exit?

yes that was what I had in mind to guard against the regression in the future, thanks!

https://github.com/llvm/llvm-project/pull/108833


More information about the llvm-commits mailing list