Skip to content

Commit 2fb3774

Browse files
committed
Revert "[SLP]Fix PR91467: Look through scalar cast, when trying to cast to another type."
This reverts commit 2475efa. Causes crashes, see comments on 2475efa.
1 parent 99052c4 commit 2fb3774

File tree

4 files changed

+17
-8
lines changed

4 files changed

+17
-8
lines changed

llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -11419,12 +11419,8 @@ Value *BoUpSLP::gather(ArrayRef<Value *> VL, Value *Root, Type *ScalarTy) {
1141911419
if (Scalar->getType() != Ty) {
1142011420
assert(Scalar->getType()->isIntegerTy() && Ty->isIntegerTy() &&
1142111421
"Expected integer types only.");
11422-
Value *V = Scalar;
11423-
if (auto *CI = dyn_cast<CastInst>(Scalar);
11424-
isa_and_nonnull<SExtInst, ZExtInst>(CI))
11425-
V = CI->getOperand(0);
1142611422
Scalar = Builder.CreateIntCast(
11427-
V, Ty, !isKnownNonNegative(Scalar, SimplifyQuery(*DL)));
11423+
Scalar, Ty, !isKnownNonNegative(Scalar, SimplifyQuery(*DL)));
1142811424
}
1142911425

1143011426
Vec = Builder.CreateInsertElement(Vec, Scalar, Builder.getInt32(Pos));

llvm/test/Transforms/SLPVectorizer/AArch64/gather-with-minbith-user.ll

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,14 @@ define void @h() {
55
; CHECK-LABEL: define void @h() {
66
; CHECK-NEXT: entry:
77
; CHECK-NEXT: [[ARRAYIDX2:%.*]] = getelementptr i8, ptr null, i64 16
8-
; CHECK-NEXT: store <8 x i16> zeroinitializer, ptr [[ARRAYIDX2]], align 2
8+
; CHECK-NEXT: [[TMP6:%.*]] = trunc i32 0 to i1
9+
; CHECK-NEXT: [[TMP0:%.*]] = insertelement <8 x i1> <i1 false, i1 false, i1 false, i1 false, i1 poison, i1 false, i1 false, i1 false>, i1 [[TMP6]], i32 4
10+
; CHECK-NEXT: [[TMP1:%.*]] = sub <8 x i1> [[TMP0]], zeroinitializer
11+
; CHECK-NEXT: [[TMP2:%.*]] = add <8 x i1> [[TMP0]], zeroinitializer
12+
; CHECK-NEXT: [[TMP3:%.*]] = shufflevector <8 x i1> [[TMP1]], <8 x i1> [[TMP2]], <8 x i32> <i32 0, i32 1, i32 2, i32 3, i32 12, i32 13, i32 14, i32 15>
13+
; CHECK-NEXT: [[TMP5:%.*]] = or <8 x i1> [[TMP3]], zeroinitializer
14+
; CHECK-NEXT: [[TMP4:%.*]] = zext <8 x i1> [[TMP5]] to <8 x i16>
15+
; CHECK-NEXT: store <8 x i16> [[TMP4]], ptr [[ARRAYIDX2]], align 2
916
; CHECK-NEXT: ret void
1017
;
1118
entry:

llvm/test/Transforms/SLPVectorizer/AArch64/user-node-not-in-bitwidths.ll

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,12 @@ define void @h() {
55
; CHECK-LABEL: define void @h() {
66
; CHECK-NEXT: entry:
77
; CHECK-NEXT: [[ARRAYIDX2:%.*]] = getelementptr i8, ptr null, i64 16
8-
; CHECK-NEXT: store <8 x i16> zeroinitializer, ptr [[ARRAYIDX2]], align 2
8+
; CHECK-NEXT: [[TMP0:%.*]] = trunc i32 0 to i1
9+
; CHECK-NEXT: [[TMP1:%.*]] = insertelement <8 x i1> <i1 false, i1 false, i1 false, i1 false, i1 poison, i1 false, i1 false, i1 false>, i1 [[TMP0]], i32 4
10+
; CHECK-NEXT: [[TMP2:%.*]] = or <8 x i1> zeroinitializer, [[TMP1]]
11+
; CHECK-NEXT: [[TMP3:%.*]] = or <8 x i1> zeroinitializer, [[TMP2]]
12+
; CHECK-NEXT: [[TMP4:%.*]] = zext <8 x i1> [[TMP3]] to <8 x i16>
13+
; CHECK-NEXT: store <8 x i16> [[TMP4]], ptr [[ARRAYIDX2]], align 2
914
; CHECK-NEXT: ret void
1015
;
1116
entry:

llvm/test/Transforms/SLPVectorizer/SystemZ/minbitwidth-root-trunc.ll

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,10 @@
44
define void @test(ptr %a, i8 %0, i16 %b.promoted.i) {
55
; CHECK-LABEL: define void @test(
66
; CHECK-SAME: ptr [[A:%.*]], i8 [[TMP0:%.*]], i16 [[B_PROMOTED_I:%.*]]) #[[ATTR0:[0-9]+]] {
7+
; CHECK-NEXT: [[TMP2:%.*]] = zext i8 [[TMP0]] to i128
78
; CHECK-NEXT: [[TMP3:%.*]] = insertelement <4 x i16> poison, i16 [[B_PROMOTED_I]], i32 0
89
; CHECK-NEXT: [[TMP4:%.*]] = shufflevector <4 x i16> [[TMP3]], <4 x i16> poison, <4 x i32> zeroinitializer
9-
; CHECK-NEXT: [[TMP5:%.*]] = zext i8 [[TMP0]] to i16
10+
; CHECK-NEXT: [[TMP5:%.*]] = trunc i128 [[TMP2]] to i16
1011
; CHECK-NEXT: [[TMP6:%.*]] = insertelement <4 x i16> poison, i16 [[TMP5]], i32 0
1112
; CHECK-NEXT: [[TMP7:%.*]] = shufflevector <4 x i16> [[TMP6]], <4 x i16> poison, <4 x i32> zeroinitializer
1213
; CHECK-NEXT: [[TMP8:%.*]] = or <4 x i16> [[TMP4]], [[TMP7]]

0 commit comments

Comments
 (0)