[llvm] c75e1e6 - [SLP] Check for instruction before casting

[via llvm-commits]

Author: Alexey Bataev
Date: 2026-05-04T07:21:51-04:00
New Revision: c75e1e62d11be965a4dca0d816a4d19a6684b043

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

LOG: [SLP] Check for instruction before casting

Values can be transformed during earlier attempts for vectorization, so
need to check if they are still instructions to avoid a compiler crash.

Fixes #195607

Reviewers: 

Pull Request: https://github.com/llvm/llvm-project/pull/195635

Added: 
    llvm/test/Transforms/SLPVectorizer/X86/post-stores-non-inst-ops.ll

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

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
index a18df9c0882ab..43faf44564c9d 100644
--- a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
+++ b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
@@ -31124,11 +31124,12 @@ bool SLPVectorizerPass::vectorizeChainsInBlock(BasicBlock *BB, BoUpSLP &R) {
       Type *ScalarTy = getValueType(PostProcessStores.front());
       if (!::isValidElementType(ScalarTy))
         return Changed;
+      auto *IF =
+          dyn_cast<Instruction>(PostProcessStores.front()->getValueOperand());
+      auto *IB =
+          dyn_cast<Instruction>(PostProcessStores.back()->getValueOperand());
       if (!NonVectReductions && PostProcessStores.size() == 2 &&
-          cast<Instruction>(PostProcessStores.front()->getValueOperand())
-                  ->getOpcode() !=
-              cast<Instruction>(PostProcessStores.back()->getValueOperand())
-                  ->getOpcode())
+          (!IF || !IB || IF->getOpcode() != IB->getOpcode()))
         return Changed;
       ScalarTy =
           IntegerType::get(ScalarTy->getContext(),

diff  --git a/llvm/test/Transforms/SLPVectorizer/X86/post-stores-non-inst-ops.ll b/llvm/test/Transforms/SLPVectorizer/X86/post-stores-non-inst-ops.ll
new file mode 100644
index 0000000000000..fb93770a7a28e
--- /dev/null
+++ b/llvm/test/Transforms/SLPVectorizer/X86/post-stores-non-inst-ops.ll
@@ -0,0 +1,18 @@
+; NOTE: Assertions have been autogenerated by utils/update_test_checks.py UTC_ARGS: --version 6
+; RUN: opt --passes=slp-vectorizer -S -mtriple=x86_64-unknown-linux-gnu < %s | FileCheck %s
+
+define i32 @test() {
+; CHECK-LABEL: define i32 @test() {
+; CHECK-NEXT:  [[ENTRY:.*:]]
+; CHECK-NEXT:    [[TMP0:%.*]] = load i8, ptr null, align 1
+; CHECK-NEXT:    store i8 [[TMP0]], ptr null, align 1
+; CHECK-NEXT:    store i32 0, ptr null, align 4
+; CHECK-NEXT:    ret i32 0
+;
+entry:
+  %0 = load i8, ptr null, align 1
+  store i8 %0, ptr null, align 1
+  %and = and i32 0, 0
+  store i32 %and, ptr null, align 4
+  ret i32 %and
+}