Lower the fcmp instruction for <4 x float> operands.

src/IceTargetLoweringX8632.def

Index: src/IceTargetLoweringX8632.def
diff --git a/src/IceTargetLoweringX8632.def b/src/IceTargetLoweringX8632.def
index b88091a7a0199dc5d8c4e1557fe47b3712fbb1aa..92f6b39fa5fe80017e117c36b32e55fcbe95500d 100644
--- a/src/IceTargetLoweringX8632.def
+++ b/src/IceTargetLoweringX8632.def
@@ -15,7 +15,7 @@
 #ifndef SUBZERO_SRC_ICETARGETLOWERINGX8632_DEF
 #define SUBZERO_SRC_ICETARGETLOWERINGX8632_DEF
 
-#define FCMPX8632_TABLE                  \
+#define SCALAR_FCMPX8632_TABLE           \
   /* val,  dflt, swap, C1,      C2 */    \
   X(False, 0,    0,    Br_None, Br_None) \
   X(Oeq,   0,    0,    Br_ne,   Br_p)    \
@@ -35,6 +35,26 @@
   X(True,  1,    0,    Br_None, Br_None) \
 //#define X(val, dflt, swap, C1, C2)
 
+#define VECTOR_FCMPX8632_TABLE      \

[Jim Stichnoth, 2014/07/24 16:16:19]

Can you just add swap and cmpps_predicate columns to the scalar x-macros table,
instead of defining a new table?

You could also consider combining the two concrete tables in
IceTargetLoweringX8632.cpp, but that's more a matter of preference.

[wala, 2014/07/24 17:13:24]

Done.

Also used a single table for IceTargetLoweringX8632.cpp.

+  /* val,  swap, cmpps predicate */ \
+  X(False, 0,    Cmpps_Invalid)     \
+  X(Oeq,   0,    Cmpps_eq)          \
+  X(Ogt,   1,    Cmpps_lt)          \
+  X(Oge,   1,    Cmpps_le)          \
+  X(Olt,   0,    Cmpps_lt)          \
+  X(Ole,   0,    Cmpps_le)          \
+  X(One,   0,    Cmpps_Invalid)     \
+  X(Ord,   0,    Cmpps_ord)         \
+  X(Ueq,   0,    Cmpps_Invalid)     \
+  X(Ugt,   0,    Cmpps_nle)         \
+  X(Uge,   0,    Cmpps_nlt)         \
+  X(Ult,   1,    Cmpps_nle)         \
+  X(Ule,   1,    Cmpps_nlt)         \
+  X(Une,   0,    Cmpps_neq)         \
+  X(Uno,   0,    Cmpps_unord)       \
+  X(True,  0,    Cmpps_Invalid)     \
+//#define X(val, swap, pred)
+
 #define ICMPX8632_TABLE                     \
   /* val, C_32,  C1_64,   C2_64,   C3_64 */ \
   X(Eq,   Br_e,  Br_None, Br_None, Br_None) \