From: Quentin Colombet <qcolombet@apple.com>
Date: Fri, 27 Jan 2017 01:13:25 +0000 (+0000)
Subject: Revert "[AArch64][LegalizerInfo] Specify the type of the initialization list."
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=cb244079cc55d7004b0369a69fa0b55aaa5508f2;p=llvm

Revert "[AArch64][LegalizerInfo] Specify the type of the initialization list."

This reverts commit r293238.
Even with that the win7 bot is still failing:
http://lab.llvm.org:8011/builders/lld-x86_64-win7/builds/3862

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@293245 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Target/AArch64/AArch64LegalizerInfo.cpp b/lib/Target/AArch64/AArch64LegalizerInfo.cpp
index 07c73171f3a..871556c8ed9 100644
--- a/lib/Target/AArch64/AArch64LegalizerInfo.cpp
+++ b/lib/Target/AArch64/AArch64LegalizerInfo.cpp
@@ -40,179 +40,173 @@ AArch64LegalizerInfo::AArch64LegalizerInfo() {
     // These operations naturally get the right answer when used on
     // GPR32, even if the actual type is narrower.
     for (auto Ty : {s32, s64, v2s32, v4s32, v2s64})
-      setAction(InstrAspect{BinOp, Ty}, Legal);
+      setAction({BinOp, Ty}, Legal);
 
     for (auto Ty : {s1, s8, s16})
-      setAction(InstrAspect{BinOp, Ty}, WidenScalar);
+      setAction({BinOp, Ty}, WidenScalar);
   }
 
-  setAction(InstrAspect{G_GEP, p0}, Legal);
-  setAction(InstrAspect{G_GEP, 1, s64}, Legal);
+  setAction({G_GEP, p0}, Legal);
+  setAction({G_GEP, 1, s64}, Legal);
 
   for (auto Ty : {s1, s8, s16, s32})
-    setAction(InstrAspect{G_GEP, 1, Ty}, WidenScalar);
+    setAction({G_GEP, 1, Ty}, WidenScalar);
 
   for (auto BinOp : {G_LSHR, G_ASHR, G_SDIV, G_UDIV}) {
     for (auto Ty : {s32, s64})
-      setAction(InstrAspect{BinOp, Ty}, Legal);
+      setAction({BinOp, Ty}, Legal);
 
     for (auto Ty : {s1, s8, s16})
-      setAction(InstrAspect{BinOp, Ty}, WidenScalar);
+      setAction({BinOp, Ty}, WidenScalar);
   }
 
   for (auto BinOp : { G_SREM, G_UREM })
     for (auto Ty : { s1, s8, s16, s32, s64 })
-      setAction(InstrAspect{BinOp, Ty}, Lower);
+      setAction({BinOp, Ty}, Lower);
 
   for (auto Op : { G_UADDE, G_USUBE, G_SADDO, G_SSUBO, G_SMULO, G_UMULO }) {
     for (auto Ty : { s32, s64 })
-      setAction(InstrAspect{Op, Ty}, Legal);
+      setAction({Op, Ty}, Legal);
 
-    setAction(InstrAspect{Op, 1, s1}, Legal);
+    setAction({Op, 1, s1}, Legal);
   }
 
   for (auto BinOp : {G_FADD, G_FSUB, G_FMUL, G_FDIV})
     for (auto Ty : {s32, s64})
-      setAction(InstrAspect{BinOp, Ty}, Legal);
+      setAction({BinOp, Ty}, Legal);
 
-  setAction(InstrAspect{G_FREM, s32}, Libcall);
-  setAction(InstrAspect{G_FREM, s64}, Libcall);
+  setAction({G_FREM, s32}, Libcall);
+  setAction({G_FREM, s64}, Libcall);
 
   for (auto MemOp : {G_LOAD, G_STORE}) {
     for (auto Ty : {s8, s16, s32, s64, p0, v2s32})
-      setAction(InstrAspect{MemOp, Ty}, Legal);
+      setAction({MemOp, Ty}, Legal);
 
-    setAction(InstrAspect{MemOp, s1}, WidenScalar);
+    setAction({MemOp, s1}, WidenScalar);
 
     // And everything's fine in addrspace 0.
-    setAction(InstrAspect{MemOp, 1, p0}, Legal);
+    setAction({MemOp, 1, p0}, Legal);
   }
 
   // Constants
   for (auto Ty : {s32, s64}) {
-    setAction(InstrAspect{TargetOpcode::G_CONSTANT, Ty}, Legal);
-    setAction(InstrAspect{TargetOpcode::G_FCONSTANT, Ty}, Legal);
+    setAction({TargetOpcode::G_CONSTANT, Ty}, Legal);
+    setAction({TargetOpcode::G_FCONSTANT, Ty}, Legal);
   }
 
-  setAction(InstrAspect{G_CONSTANT, p0}, Legal);
+  setAction({G_CONSTANT, p0}, Legal);
 
   for (auto Ty : {s1, s8, s16})
-    setAction(InstrAspect{TargetOpcode::G_CONSTANT, Ty}, WidenScalar);
+    setAction({TargetOpcode::G_CONSTANT, Ty}, WidenScalar);
 
-  setAction(InstrAspect{TargetOpcode::G_FCONSTANT, s16}, WidenScalar);
+  setAction({TargetOpcode::G_FCONSTANT, s16}, WidenScalar);
 
-  setAction(InstrAspect{G_ICMP, s1}, Legal);
-  setAction(InstrAspect{G_ICMP, 1, s32}, Legal);
-  setAction(InstrAspect{G_ICMP, 1, s64}, Legal);
-  setAction(InstrAspect{G_ICMP, 1, p0}, Legal);
+  setAction({G_ICMP, s1}, Legal);
+  setAction({G_ICMP, 1, s32}, Legal);
+  setAction({G_ICMP, 1, s64}, Legal);
+  setAction({G_ICMP, 1, p0}, Legal);
 
   for (auto Ty : {s1, s8, s16}) {
-    setAction(InstrAspect{G_ICMP, 1, Ty}, WidenScalar);
+    setAction({G_ICMP, 1, Ty}, WidenScalar);
   }
 
-  setAction(InstrAspect{G_FCMP, s1}, Legal);
-  setAction(InstrAspect{G_FCMP, 1, s32}, Legal);
-  setAction(InstrAspect{G_FCMP, 1, s64}, Legal);
+  setAction({G_FCMP, s1}, Legal);
+  setAction({G_FCMP, 1, s32}, Legal);
+  setAction({G_FCMP, 1, s64}, Legal);
 
   // Extensions
   for (auto Ty : { s1, s8, s16, s32, s64 }) {
-    setAction(InstrAspect{G_ZEXT, Ty}, Legal);
-    setAction(InstrAspect{G_SEXT, Ty}, Legal);
-    setAction(InstrAspect{G_ANYEXT, Ty}, Legal);
+    setAction({G_ZEXT, Ty}, Legal);
+    setAction({G_SEXT, Ty}, Legal);
+    setAction({G_ANYEXT, Ty}, Legal);
   }
 
   for (auto Ty : { s1, s8, s16, s32 }) {
-    setAction(InstrAspect{G_ZEXT, 1, Ty}, Legal);
-    setAction(InstrAspect{G_SEXT, 1, Ty}, Legal);
-    setAction(InstrAspect{G_ANYEXT, 1, Ty}, Legal);
+    setAction({G_ZEXT, 1, Ty}, Legal);
+    setAction({G_SEXT, 1, Ty}, Legal);
+    setAction({G_ANYEXT, 1, Ty}, Legal);
   }
 
-  setAction(InstrAspect{G_FPEXT, s64}, Legal);
-  setAction(InstrAspect{G_FPEXT, 1, s32}, Legal);
+  setAction({G_FPEXT, s64}, Legal);
+  setAction({G_FPEXT, 1, s32}, Legal);
 
   // Truncations
   for (auto Ty : { s16, s32 })
-    setAction(InstrAspect{G_FPTRUNC, Ty}, Legal);
+    setAction({G_FPTRUNC, Ty}, Legal);
 
   for (auto Ty : { s32, s64 })
-    setAction(InstrAspect{G_FPTRUNC, 1, Ty}, Legal);
+    setAction({G_FPTRUNC, 1, Ty}, Legal);
 
   for (auto Ty : { s1, s8, s16, s32 })
-    setAction(InstrAspect{G_TRUNC, Ty}, Legal);
+    setAction({G_TRUNC, Ty}, Legal);
 
   for (auto Ty : { s8, s16, s32, s64 })
-    setAction(InstrAspect{G_TRUNC, 1, Ty}, Legal);
+    setAction({G_TRUNC, 1, Ty}, Legal);
 
   // Conversions
   for (auto Ty : { s32, s64 }) {
-    setAction(InstrAspect{G_FPTOSI, 0, Ty}, Legal);
-    setAction(InstrAspect{G_FPTOUI, 0, Ty}, Legal);
-    setAction(InstrAspect{G_SITOFP, 1, Ty}, Legal);
-    setAction(InstrAspect{G_UITOFP, 1, Ty}, Legal);
+    setAction({G_FPTOSI, 0, Ty}, Legal);
+    setAction({G_FPTOUI, 0, Ty}, Legal);
+    setAction({G_SITOFP, 1, Ty}, Legal);
+    setAction({G_UITOFP, 1, Ty}, Legal);
   }
   for (auto Ty : { s1, s8, s16 }) {
-    setAction(InstrAspect{G_FPTOSI, 0, Ty}, WidenScalar);
-    setAction(InstrAspect{G_FPTOUI, 0, Ty}, WidenScalar);
-    setAction(InstrAspect{G_SITOFP, 1, Ty}, WidenScalar);
-    setAction(InstrAspect{G_UITOFP, 1, Ty}, WidenScalar);
+    setAction({G_FPTOSI, 0, Ty}, WidenScalar);
+    setAction({G_FPTOUI, 0, Ty}, WidenScalar);
+    setAction({G_SITOFP, 1, Ty}, WidenScalar);
+    setAction({G_UITOFP, 1, Ty}, WidenScalar);
   }
 
   for (auto Ty : { s32, s64 }) {
-    setAction(InstrAspect{G_FPTOSI, 1, Ty}, Legal);
-    setAction(InstrAspect{G_FPTOUI, 1, Ty}, Legal);
-    setAction(InstrAspect{G_SITOFP, 0, Ty}, Legal);
-    setAction(InstrAspect{G_UITOFP, 0, Ty}, Legal);
+    setAction({G_FPTOSI, 1, Ty}, Legal);
+    setAction({G_FPTOUI, 1, Ty}, Legal);
+    setAction({G_SITOFP, 0, Ty}, Legal);
+    setAction({G_UITOFP, 0, Ty}, Legal);
   }
 
   // Control-flow
   for (auto Ty : {s1, s8, s16, s32})
-    setAction(InstrAspect{G_BRCOND, Ty}, Legal);
+    setAction({G_BRCOND, Ty}, Legal);
 
   // Select
   for (auto Ty : {s1, s8, s16, s32, s64, p0})
-    setAction(InstrAspect{G_SELECT, Ty}, Legal);
+    setAction({G_SELECT, Ty}, Legal);
 
-  setAction(InstrAspect{G_SELECT, 1, s1}, Legal);
+  setAction({G_SELECT, 1, s1}, Legal);
 
   // Pointer-handling
-  setAction(InstrAspect{G_FRAME_INDEX, p0}, Legal);
-  setAction(InstrAspect{G_GLOBAL_VALUE, p0}, Legal);
+  setAction({G_FRAME_INDEX, p0}, Legal);
+  setAction({G_GLOBAL_VALUE, p0}, Legal);
 
   for (auto Ty : {s1, s8, s16, s32, s64})
-    setAction(InstrAspect{G_PTRTOINT, 0, Ty}, Legal);
+    setAction({G_PTRTOINT, 0, Ty}, Legal);
 
-  setAction(InstrAspect{G_PTRTOINT, 1, p0}, Legal);
+  setAction({G_PTRTOINT, 1, p0}, Legal);
 
-  setAction(InstrAspect{G_INTTOPTR, 0, p0}, Legal);
-  setAction(InstrAspect{G_INTTOPTR, 1, s64}, Legal);
+  setAction({G_INTTOPTR, 0, p0}, Legal);
+  setAction({G_INTTOPTR, 1, s64}, Legal);
 
   // Casts for 32 and 64-bit width type are just copies.
   for (auto Ty : {s1, s8, s16, s32, s64}) {
-    setAction(InstrAspect{G_BITCAST, 0, Ty}, Legal);
-    setAction(InstrAspect{G_BITCAST, 1, Ty}, Legal);
+    setAction({G_BITCAST, 0, Ty}, Legal);
+    setAction({G_BITCAST, 1, Ty}, Legal);
   }
 
   // For the sake of copying bits around, the type does not really
   // matter as long as it fits a register.
   for (int EltSize = 8; EltSize <= 64; EltSize *= 2) {
-    setAction(InstrAspect{G_BITCAST, 0, LLT::vector(128 / EltSize, EltSize)},
-              Legal);
-    setAction(InstrAspect{G_BITCAST, 1, LLT::vector(128 / EltSize, EltSize)},
-              Legal);
+    setAction({G_BITCAST, 0, LLT::vector(128/EltSize, EltSize)}, Legal);
+    setAction({G_BITCAST, 1, LLT::vector(128/EltSize, EltSize)}, Legal);
     if (EltSize >= 64)
       continue;
 
-    setAction(InstrAspect{G_BITCAST, 0, LLT::vector(64 / EltSize, EltSize)},
-              Legal);
-    setAction(InstrAspect{G_BITCAST, 1, LLT::vector(64 / EltSize, EltSize)},
-              Legal);
+    setAction({G_BITCAST, 0, LLT::vector(64/EltSize, EltSize)}, Legal);
+    setAction({G_BITCAST, 1, LLT::vector(64/EltSize, EltSize)}, Legal);
     if (EltSize >= 32)
       continue;
 
-    setAction(InstrAspect{G_BITCAST, 0, LLT::vector(32 / EltSize, EltSize)},
-              Legal);
-    setAction(InstrAspect{G_BITCAST, 1, LLT::vector(32 / EltSize, EltSize)},
-              Legal);
+    setAction({G_BITCAST, 0, LLT::vector(32/EltSize, EltSize)}, Legal);
+    setAction({G_BITCAST, 1, LLT::vector(32/EltSize, EltSize)}, Legal);
   }
 
   computeTables();