[llvm] e36ddff - [VPlan] Add scalable check to SinkStoreInfo helper.

[Florian Hahn via llvm-commits]

Author: Florian Hahn
Date: 2026-01-16T21:07:40Z
New Revision: e36ddff7a475bd18d68fbdaa6e4a8ad924f85490

URL: https://github.com/llvm/llvm-project/commit/e36ddff7a475bd18d68fbdaa6e4a8ad924f85490
DIFF: https://github.com/llvm/llvm-project/commit/e36ddff7a475bd18d68fbdaa6e4a8ad924f85490.diff

LOG: [VPlan] Add scalable check to SinkStoreInfo helper.

Bail out on scalable vectors in helper. Currently this is not causing
issues, but fixes a potential crash that would be exposed by a follow-up
change.

Test would exposes the issue in the future has been added in
8c5352cf3e14ec0c56f592091899d229de8436a7.

Added: 
    

Modified: 
    llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp b/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
index 4e876b256f04d..a39b171ab4cd6 100644
--- a/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
+++ b/llvm/lib/Transforms/Vectorize/VPlanTransforms.cpp
@@ -163,6 +163,8 @@ class SinkStoreInfo {
 
     auto VFs = B->getParent()->getPlan()->vectorFactors();
     ElementCount MaxVF = *max_element(VFs, ElementCount::isKnownLT);
+    if (MaxVF.isScalable())
+      return false;
     return Distance->abs().uge(
         MaxVF.multiplyCoefficientBy(MaxStoreSize).getFixedValue());
   }