Skip to content

Commit 273f4de

Browse files
diggerlindiggerlin
committed
return POISON value in the getNode() in input is poison
1 parent 5aeeebc commit 273f4de

File tree

4 files changed

+25
-31
lines changed

4 files changed

+25
-31
lines changed

‎llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp‎

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7601,16 +7601,23 @@ SDValue SelectionDAG::getNode(unsigned Opcode, const SDLoc &DL, EVT VT,
76017601
case ISD::SDIV:
76027602
case ISD::UREM:
76037603
case ISD::SREM:
7604-
return getUNDEF(VT); // fold op(arg1, undef) -> undef
7604+
// fold op(arg1, undef) -> undef, // fold op(arg1, poison) -> poison.
7605+
return N2.getOpcode() == ISD::POISON
7606+
? getPOISON(VT)
7607+
: getUNDEF(VT);
76057608
case ISD::MUL:
76067609
case ISD::AND:
76077610
case ISD::SSUBSAT:
76087611
case ISD::USUBSAT:
7609-
return getConstant(0, DL, VT); // fold op(arg1, undef) -> 0
7612+
// fold op(arg1, undef) -> 0, fold op(arg1, poison) -> poison.
7613+
return N2.getOpcode() == ISD::POISON
7614+
? getPOISON(VT)
7615+
: getConstant(0, DL, VT);
76107616
case ISD::OR:
76117617
case ISD::SADDSAT:
76127618
case ISD::UADDSAT:
7613-
return getAllOnesConstant(DL, VT);
7619+
return N2.getOpcode() == ISD::POISON ? getPOISON(VT)
7620+
: getAllOnesConstant(DL, VT);
76147621
}
76157622
}
76167623

‎llvm/test/CodeGen/X86/half.ll‎

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1991,8 +1991,8 @@ define void @pr63114() {
19911991
; CHECK-LIBCALL-LABEL: pr63114:
19921992
; CHECK-LIBCALL: # %bb.0:
19931993
; CHECK-LIBCALL-NEXT: movdqu (%rax), %xmm4
1994-
; CHECK-LIBCALL-NEXT: pshufhw {{.*#+}} xmm0 = xmm4[0,1,2,3,4,5,7,7]
1995-
; CHECK-LIBCALL-NEXT: pshufd {{.*#+}} xmm0 = xmm0[0,2,2,3]
1994+
; CHECK-LIBCALL-NEXT: pshuflw {{.*#+}} xmm0 = xmm4[0,1,3,3,4,5,6,7]
1995+
; CHECK-LIBCALL-NEXT: pshufd {{.*#+}} xmm0 = xmm0[0,0,2,1]
19961996
; CHECK-LIBCALL-NEXT: movdqa {{.*#+}} xmm1 = [65535,65535,65535,0,65535,65535,65535,65535]
19971997
; CHECK-LIBCALL-NEXT: pand %xmm1, %xmm0
19981998
; CHECK-LIBCALL-NEXT: movq {{.*#+}} xmm2 = [0,0,0,15360,0,0,0,0]
@@ -2001,8 +2001,8 @@ define void @pr63114() {
20012001
; CHECK-LIBCALL-NEXT: pand %xmm3, %xmm0
20022002
; CHECK-LIBCALL-NEXT: movdqa {{.*#+}} xmm5 = [0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,60]
20032003
; CHECK-LIBCALL-NEXT: por %xmm5, %xmm0
2004-
; CHECK-LIBCALL-NEXT: pshuflw {{.*#+}} xmm6 = xmm4[0,1,3,3,4,5,6,7]
2005-
; CHECK-LIBCALL-NEXT: pshufd {{.*#+}} xmm6 = xmm6[0,0,2,1]
2004+
; CHECK-LIBCALL-NEXT: pshufhw {{.*#+}} xmm6 = xmm4[0,1,2,3,4,5,7,7]
2005+
; CHECK-LIBCALL-NEXT: pshufd {{.*#+}} xmm6 = xmm6[0,2,2,3]
20062006
; CHECK-LIBCALL-NEXT: pand %xmm1, %xmm6
20072007
; CHECK-LIBCALL-NEXT: por %xmm2, %xmm6
20082008
; CHECK-LIBCALL-NEXT: pand %xmm3, %xmm6
@@ -2020,8 +2020,8 @@ define void @pr63114() {
20202020
; CHECK-LIBCALL-NEXT: por %xmm5, %xmm7
20212021
; CHECK-LIBCALL-NEXT: movdqu %xmm7, 0
20222022
; CHECK-LIBCALL-NEXT: movdqu %xmm4, 32
2023-
; CHECK-LIBCALL-NEXT: movdqu %xmm6, 16
2024-
; CHECK-LIBCALL-NEXT: movdqu %xmm0, 48
2023+
; CHECK-LIBCALL-NEXT: movdqu %xmm6, 48
2024+
; CHECK-LIBCALL-NEXT: movdqu %xmm0, 16
20252025
; CHECK-LIBCALL-NEXT: retq
20262026
;
20272027
; BWON-F16C-LABEL: pr63114:
@@ -2056,8 +2056,8 @@ define void @pr63114() {
20562056
; CHECK-I686-LABEL: pr63114:
20572057
; CHECK-I686: # %bb.0:
20582058
; CHECK-I686-NEXT: movdqu (%eax), %xmm6
2059-
; CHECK-I686-NEXT: pshufhw {{.*#+}} xmm0 = xmm6[0,1,2,3,4,5,7,7]
2060-
; CHECK-I686-NEXT: pshufd {{.*#+}} xmm0 = xmm0[0,2,2,3]
2059+
; CHECK-I686-NEXT: pshuflw {{.*#+}} xmm0 = xmm6[0,1,3,3,4,5,6,7]
2060+
; CHECK-I686-NEXT: pshufd {{.*#+}} xmm0 = xmm0[0,0,2,1]
20612061
; CHECK-I686-NEXT: movdqa {{.*#+}} xmm1 = [65535,65535,65535,0,65535,65535,65535,65535]
20622062
; CHECK-I686-NEXT: pand %xmm1, %xmm0
20632063
; CHECK-I686-NEXT: movq {{.*#+}} xmm2 = [0,0,0,15360,0,0,0,0]
@@ -2066,8 +2066,8 @@ define void @pr63114() {
20662066
; CHECK-I686-NEXT: pand %xmm3, %xmm0
20672067
; CHECK-I686-NEXT: movdqa {{.*#+}} xmm4 = [0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,60]
20682068
; CHECK-I686-NEXT: por %xmm4, %xmm0
2069-
; CHECK-I686-NEXT: pshuflw {{.*#+}} xmm5 = xmm6[0,1,3,3,4,5,6,7]
2070-
; CHECK-I686-NEXT: pshufd {{.*#+}} xmm5 = xmm5[0,0,2,1]
2069+
; CHECK-I686-NEXT: pshufhw {{.*#+}} xmm5 = xmm6[0,1,2,3,4,5,7,7]
2070+
; CHECK-I686-NEXT: pshufd {{.*#+}} xmm5 = xmm5[0,2,2,3]
20712071
; CHECK-I686-NEXT: pand %xmm1, %xmm5
20722072
; CHECK-I686-NEXT: por %xmm2, %xmm5
20732073
; CHECK-I686-NEXT: pand %xmm3, %xmm5
@@ -2085,8 +2085,8 @@ define void @pr63114() {
20852085
; CHECK-I686-NEXT: por %xmm4, %xmm7
20862086
; CHECK-I686-NEXT: movdqu %xmm7, 0
20872087
; CHECK-I686-NEXT: movdqu %xmm6, 32
2088-
; CHECK-I686-NEXT: movdqu %xmm5, 16
2089-
; CHECK-I686-NEXT: movdqu %xmm0, 48
2088+
; CHECK-I686-NEXT: movdqu %xmm5, 48
2089+
; CHECK-I686-NEXT: movdqu %xmm0, 16
20902090
; CHECK-I686-NEXT: retl
20912091
%1 = load <24 x half>, ptr poison, align 2
20922092
%2 = shufflevector <24 x half> %1, <24 x half> poison, <8 x i32> <i32 2, i32 5, i32 8, i32 11, i32 14, i32 17, i32 20, i32 23>

‎llvm/test/CodeGen/X86/poison-ops.ll‎

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,6 @@ define <4 x i32> @sub_poison_lhs_vec(<4 x i32> %x) {
6868
define i32 @mul_poison_rhs(i32 %x) {
6969
; CHECK-LABEL: mul_poison_rhs:
7070
; CHECK: # %bb.0:
71-
; CHECK-NEXT: xorl %eax, %eax
7271
; CHECK-NEXT: retq
7372
%r = mul i32 %x, poison
7473
ret i32 %r
@@ -77,7 +76,6 @@ define i32 @mul_poison_rhs(i32 %x) {
7776
define <4 x i32> @mul_poison_rhs_vec(<4 x i32> %x) {
7877
; CHECK-LABEL: mul_poison_rhs_vec:
7978
; CHECK: # %bb.0:
80-
; CHECK-NEXT: xorps %xmm0, %xmm0
8179
; CHECK-NEXT: retq
8280
%r = mul <4 x i32> %x, poison
8381
ret <4 x i32> %r
@@ -86,7 +84,6 @@ define <4 x i32> @mul_poison_rhs_vec(<4 x i32> %x) {
8684
define i32 @mul_poison_lhs(i32 %x) {
8785
; CHECK-LABEL: mul_poison_lhs:
8886
; CHECK: # %bb.0:
89-
; CHECK-NEXT: xorl %eax, %eax
9087
; CHECK-NEXT: retq
9188
%r = mul i32 poison, %x
9289
ret i32 %r
@@ -95,7 +92,6 @@ define i32 @mul_poison_lhs(i32 %x) {
9592
define <4 x i32> @mul_poison_lhs_vec(<4 x i32> %x) {
9693
; CHECK-LABEL: mul_poison_lhs_vec:
9794
; CHECK: # %bb.0:
98-
; CHECK-NEXT: xorps %xmm0, %xmm0
9995
; CHECK-NEXT: retq
10096
%r = mul <4 x i32> poison, %x
10197
ret <4 x i32> %r
@@ -342,7 +338,6 @@ define <4 x i32> @shl_poison_lhs_vec(<4 x i32> %x) {
342338
define i32 @and_poison_rhs(i32 %x) {
343339
; CHECK-LABEL: and_poison_rhs:
344340
; CHECK: # %bb.0:
345-
; CHECK-NEXT: xorl %eax, %eax
346341
; CHECK-NEXT: retq
347342
%r = and i32 %x, poison
348343
ret i32 %r
@@ -351,7 +346,6 @@ define i32 @and_poison_rhs(i32 %x) {
351346
define <4 x i32> @and_poison_rhs_vec(<4 x i32> %x) {
352347
; CHECK-LABEL: and_poison_rhs_vec:
353348
; CHECK: # %bb.0:
354-
; CHECK-NEXT: xorps %xmm0, %xmm0
355349
; CHECK-NEXT: retq
356350
%r = and <4 x i32> %x, poison
357351
ret <4 x i32> %r
@@ -360,7 +354,6 @@ define <4 x i32> @and_poison_rhs_vec(<4 x i32> %x) {
360354
define i32 @and_poison_lhs(i32 %x) {
361355
; CHECK-LABEL: and_poison_lhs:
362356
; CHECK: # %bb.0:
363-
; CHECK-NEXT: xorl %eax, %eax
364357
; CHECK-NEXT: retq
365358
%r = and i32 poison, %x
366359
ret i32 %r
@@ -369,7 +362,6 @@ define i32 @and_poison_lhs(i32 %x) {
369362
define <4 x i32> @and_poison_lhs_vec(<4 x i32> %x) {
370363
; CHECK-LABEL: and_poison_lhs_vec:
371364
; CHECK: # %bb.0:
372-
; CHECK-NEXT: xorps %xmm0, %xmm0
373365
; CHECK-NEXT: retq
374366
%r = and <4 x i32> poison, %x
375367
ret <4 x i32> %r
@@ -378,7 +370,6 @@ define <4 x i32> @and_poison_lhs_vec(<4 x i32> %x) {
378370
define i32 @or_poison_rhs(i32 %x) {
379371
; CHECK-LABEL: or_poison_rhs:
380372
; CHECK: # %bb.0:
381-
; CHECK-NEXT: movl $-1, %eax
382373
; CHECK-NEXT: retq
383374
%r = or i32 %x, poison
384375
ret i32 %r
@@ -387,7 +378,6 @@ define i32 @or_poison_rhs(i32 %x) {
387378
define <4 x i32> @or_poison_rhs_vec(<4 x i32> %x) {
388379
; CHECK-LABEL: or_poison_rhs_vec:
389380
; CHECK: # %bb.0:
390-
; CHECK-NEXT: pcmpeqd %xmm0, %xmm0
391381
; CHECK-NEXT: retq
392382
%r = or <4 x i32> %x, poison
393383
ret <4 x i32> %r
@@ -396,7 +386,6 @@ define <4 x i32> @or_poison_rhs_vec(<4 x i32> %x) {
396386
define i32 @or_poison_lhs(i32 %x) {
397387
; CHECK-LABEL: or_poison_lhs:
398388
; CHECK: # %bb.0:
399-
; CHECK-NEXT: movl $-1, %eax
400389
; CHECK-NEXT: retq
401390
%r = or i32 poison, %x
402391
ret i32 %r
@@ -405,7 +394,6 @@ define i32 @or_poison_lhs(i32 %x) {
405394
define <4 x i32> @or_poison_lhs_vec(<4 x i32> %x) {
406395
; CHECK-LABEL: or_poison_lhs_vec:
407396
; CHECK: # %bb.0:
408-
; CHECK-NEXT: pcmpeqd %xmm0, %xmm0
409397
; CHECK-NEXT: retq
410398
%r = or <4 x i32> poison, %x
411399
ret <4 x i32> %r

‎llvm/test/CodeGen/X86/pr119158.ll‎

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,9 @@ define dso_local void @foo() #1 {
55
; CHECK-LABEL: foo:
66
; CHECK: # %bb.0: # %newFuncRoot
77
; CHECK-NEXT: vpmovzxbd {{.*#+}} ymm0 = mem[0],zero,zero,zero,mem[1],zero,zero,zero,mem[2],zero,zero,zero,mem[3],zero,zero,zero,mem[4],zero,zero,zero,mem[5],zero,zero,zero,mem[6],zero,zero,zero,mem[7],zero,zero,zero
8-
; CHECK-NEXT: vpbroadcastd {{.*#+}} ymm1 = [18,0,18,0,18,0,18,0,18,0,18,0,18,0,18,0]
9-
; CHECK-NEXT: vpmaddwd %ymm1, %ymm0, %ymm0
10-
; CHECK-NEXT: vpaddd {{\.?LCPI[0-9]+_[0-9]+}}(%rip){1to8}, %ymm0, %ymm0
11-
; CHECK-NEXT: vpsrld $7, %ymm0, %ymm0
8+
; CHECK-NEXT: vpbroadcastd {{.*#+}} ymm1 = [64,64,64,64,64,64,64,64]
9+
; CHECK-NEXT: vpdpwssd {{\.?LCPI[0-9]+_[0-9]+}}(%rip){1to8}, %ymm0, %ymm1
10+
; CHECK-NEXT: vpsrld $7, %ymm1, %ymm0
1211
; CHECK-NEXT: vpackusdw %ymm0, %ymm0, %ymm0
1312
; CHECK-NEXT: vpermq {{.*#+}} ymm0 = ymm0[0,2,1,3]
1413
; CHECK-NEXT: vmovdqu %ymm0, (%rax)

0 commit comments

Comments
 (0)