summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorCraig Topper <craig.topper@intel.com>2018-06-22 21:09:31 +0000
committerCraig Topper <craig.topper@intel.com>2018-06-22 21:09:31 +0000
commita55cc4a2e94a869676417ae8a4bce92bcaf82399 (patch)
tree9cd67959d7c602752b18c37b0480c72d73895b43
parente2a7f3106446f644372ba920bdcd9c16a8cc3b02 (diff)
downloadbcm5719-llvm-a55cc4a2e94a869676417ae8a4bce92bcaf82399.tar.gz
bcm5719-llvm-a55cc4a2e94a869676417ae8a4bce92bcaf82399.zip
[X86] Add test cases showing missed select simplifcation for MCU when icmp is in a slightly different form.
These test cases show that the "(select (and (x , 0x1) == 0), y, (z ^ y) ) -> (-(and (x , 0x1)) & z ) ^ y" doesn't work if the select condition is changed to (and (x, 0x1) != 1) llvm-svn: 335389
-rw-r--r--llvm/test/CodeGen/X86/select.ll110
1 files changed, 110 insertions, 0 deletions
diff --git a/llvm/test/CodeGen/X86/select.ll b/llvm/test/CodeGen/X86/select.ll
index 7e881de0c80..4fc88d56a67 100644
--- a/llvm/test/CodeGen/X86/select.ll
+++ b/llvm/test/CodeGen/X86/select.ll
@@ -1102,6 +1102,35 @@ entry:
ret i16 %1
}
+; Equivalent to above, but with icmp ne (and %cond, 1), 1 instead of
+; icmp eq (and %cond, 1), 0
+define i16 @select_xor_1b(i16 %A, i8 %cond) {
+; CHECK-LABEL: select_xor_1b:
+; CHECK: ## %bb.0: ## %entry
+; CHECK-NEXT: movl %edi, %eax
+; CHECK-NEXT: xorl $43, %eax
+; CHECK-NEXT: testb $1, %sil
+; CHECK-NEXT: cmovel %edi, %eax
+; CHECK-NEXT: ## kill: def $ax killed $ax killed $eax
+; CHECK-NEXT: retq
+;
+; MCU-LABEL: select_xor_1b:
+; MCU: # %bb.0: # %entry
+; MCU-NEXT: testb $1, %dl
+; MCU-NEXT: je .LBB26_2
+; MCU-NEXT: # %bb.1:
+; MCU-NEXT: xorl $43, %eax
+; MCU-NEXT: .LBB26_2: # %entry
+; MCU-NEXT: # kill: def $ax killed $ax killed $eax
+; MCU-NEXT: retl
+entry:
+ %and = and i8 %cond, 1
+ %cmp10 = icmp ne i8 %and, 1
+ %0 = xor i16 %A, 43
+ %1 = select i1 %cmp10, i16 %A, i16 %0
+ ret i16 %1
+}
+
define i32 @select_xor_2(i32 %A, i32 %B, i8 %cond) {
; CHECK-LABEL: select_xor_2:
; CHECK: ## %bb.0: ## %entry
@@ -1126,6 +1155,33 @@ entry:
ret i32 %1
}
+; Equivalent to above, but with icmp ne (and %cond, 1), 1 instead of
+; icmp eq (and %cond, 1), 0
+define i32 @select_xor_2b(i32 %A, i32 %B, i8 %cond) {
+; CHECK-LABEL: select_xor_2b:
+; CHECK: ## %bb.0: ## %entry
+; CHECK-NEXT: xorl %edi, %esi
+; CHECK-NEXT: testb $1, %dl
+; CHECK-NEXT: cmovel %edi, %esi
+; CHECK-NEXT: movl %esi, %eax
+; CHECK-NEXT: retq
+;
+; MCU-LABEL: select_xor_2b:
+; MCU: # %bb.0: # %entry
+; MCU-NEXT: testb $1, %cl
+; MCU-NEXT: je .LBB28_2
+; MCU-NEXT: # %bb.1:
+; MCU-NEXT: xorl %edx, %eax
+; MCU-NEXT: .LBB28_2: # %entry
+; MCU-NEXT: retl
+entry:
+ %and = and i8 %cond, 1
+ %cmp10 = icmp ne i8 %and, 1
+ %0 = xor i32 %B, %A
+ %1 = select i1 %cmp10, i32 %A, i32 %0
+ ret i32 %1
+}
+
define i32 @select_or(i32 %A, i32 %B, i8 %cond) {
; CHECK-LABEL: select_or:
; CHECK: ## %bb.0: ## %entry
@@ -1150,6 +1206,33 @@ entry:
ret i32 %1
}
+; Equivalent to above, but with icmp ne (and %cond, 1), 1 instead of
+; icmp eq (and %cond, 1), 0
+define i32 @select_or_b(i32 %A, i32 %B, i8 %cond) {
+; CHECK-LABEL: select_or_b:
+; CHECK: ## %bb.0: ## %entry
+; CHECK-NEXT: orl %edi, %esi
+; CHECK-NEXT: testb $1, %dl
+; CHECK-NEXT: cmovel %edi, %esi
+; CHECK-NEXT: movl %esi, %eax
+; CHECK-NEXT: retq
+;
+; MCU-LABEL: select_or_b:
+; MCU: # %bb.0: # %entry
+; MCU-NEXT: testb $1, %cl
+; MCU-NEXT: je .LBB30_2
+; MCU-NEXT: # %bb.1:
+; MCU-NEXT: orl %edx, %eax
+; MCU-NEXT: .LBB30_2: # %entry
+; MCU-NEXT: retl
+entry:
+ %and = and i8 %cond, 1
+ %cmp10 = icmp ne i8 %and, 1
+ %0 = or i32 %B, %A
+ %1 = select i1 %cmp10, i32 %A, i32 %0
+ ret i32 %1
+}
+
define i32 @select_or_1(i32 %A, i32 %B, i32 %cond) {
; CHECK-LABEL: select_or_1:
; CHECK: ## %bb.0: ## %entry
@@ -1173,3 +1256,30 @@ entry:
%1 = select i1 %cmp10, i32 %A, i32 %0
ret i32 %1
}
+
+; Equivalent to above, but with icmp ne (and %cond, 1), 1 instead of
+; icmp eq (and %cond, 1), 0
+define i32 @select_or_1b(i32 %A, i32 %B, i32 %cond) {
+; CHECK-LABEL: select_or_1b:
+; CHECK: ## %bb.0: ## %entry
+; CHECK-NEXT: orl %edi, %esi
+; CHECK-NEXT: testb $1, %dl
+; CHECK-NEXT: cmovel %edi, %esi
+; CHECK-NEXT: movl %esi, %eax
+; CHECK-NEXT: retq
+;
+; MCU-LABEL: select_or_1b:
+; MCU: # %bb.0: # %entry
+; MCU-NEXT: testb $1, %cl
+; MCU-NEXT: je .LBB32_2
+; MCU-NEXT: # %bb.1:
+; MCU-NEXT: orl %edx, %eax
+; MCU-NEXT: .LBB32_2: # %entry
+; MCU-NEXT: retl
+entry:
+ %and = and i32 %cond, 1
+ %cmp10 = icmp ne i32 %and, 1
+ %0 = or i32 %B, %A
+ %1 = select i1 %cmp10, i32 %A, i32 %0
+ ret i32 %1
+}
OpenPOWER on IntegriCloud