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

David Sherwood via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 27 05:18:48 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())) {
----------------
david-arm wrote:

You're absolutely right. I was being too eager! I looked and there is no defence against loops with uncountable exits, since LoopVectorizationLegality doesn't filter those out in advance. I've reverted this change.

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


More information about the llvm-commits mailing list