Subzero: Fix some legalization issues involving immediates.

tests_lit/llvm2ice_tests/bool-opt.ll

Index: tests_lit/llvm2ice_tests/bool-opt.ll
diff --git a/tests_lit/llvm2ice_tests/bool-opt.ll b/tests_lit/llvm2ice_tests/bool-opt.ll
index 40cc44b193e9dc5129117aba31eb050fd4cfd324..92d8d44d5b2461d9b3ff3d6f9cf295abedf179e0 100644
--- a/tests_lit/llvm2ice_tests/bool-opt.ll
+++ b/tests_lit/llvm2ice_tests/bool-opt.ll
@@ -14,6 +14,18 @@ entry:
   ret void
 }
 
+; Check that correct addressing modes are used for comparing two
+; immediates.
+define void @testIcmpImm() {
+entry:
+  %cmp = icmp eq i32 1, 2
+  %cmp_ext = zext i1 %cmp to i32
+  tail call void @use(i32 %cmp_ext)
+  ret void
+}
+; CHECK-LABEL: testIcmpImm
+; CHECK-NOT: cmp {{[0-9]+}}, {{[0-9]+}}

[jvoung (off chromium), 2014/08/26 21:59:46]

Do you need the second pair of ints after the comma (the other tests just check
the first src)

[Jim Stichnoth, 2014/08/27 06:19:47]

Done.

+
 declare void @use(i32)
 
 ; CHECK-NOT: ICE translation error