[llvm] r293245 - Revert "[AArch64][LegalizerInfo] Specify the type of the initialization list."
Quentin Colombet via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 26 17:13:26 PST 2017
Author: qcolombet
Date: Thu Jan 26 19:13:25 2017
New Revision: 293245
URL: http://llvm.org/viewvc/llvm-project?rev=293245&view=rev
Log:
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
Modified:
llvm/trunk/lib/Target/AArch64/AArch64LegalizerInfo.cpp
Modified: llvm/trunk/lib/Target/AArch64/AArch64LegalizerInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AArch64/AArch64LegalizerInfo.cpp?rev=293245&r1=293244&r2=293245&view=diff
==============================================================================
--- llvm/trunk/lib/Target/AArch64/AArch64LegalizerInfo.cpp (original)
+++ llvm/trunk/lib/Target/AArch64/AArch64LegalizerInfo.cpp Thu Jan 26 19:13:25 2017
@@ -40,179 +40,173 @@ AArch64LegalizerInfo::AArch64LegalizerIn
// 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();
More information about the llvm-commits
mailing list