[llvm] r309163 - [GISel]: Avoid zero length array when building Instrs that don't have
Aditya Nandakumar via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 26 12:58:03 PDT 2017
Author: aditya_nandakumar
Date: Wed Jul 26 12:58:03 2017
New Revision: 309163
URL: http://llvm.org/viewvc/llvm-project?rev=309163&view=rev
Log:
[GISel]: Avoid zero length array when building Instrs that don't have
uses.
Also splitting the buildSources part allows more overloads such as
adding MachineOperands directly in the arguments for buildInstr.
Modified:
llvm/trunk/include/llvm/CodeGen/GlobalISel/MachineIRBuilder.h
Modified: llvm/trunk/include/llvm/CodeGen/GlobalISel/MachineIRBuilder.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/CodeGen/GlobalISel/MachineIRBuilder.h?rev=309163&r1=309162&r2=309163&view=diff
==============================================================================
--- llvm/trunk/include/llvm/CodeGen/GlobalISel/MachineIRBuilder.h (original)
+++ llvm/trunk/include/llvm/CodeGen/GlobalISel/MachineIRBuilder.h Wed Jul 26 12:58:03 2017
@@ -70,12 +70,20 @@ class MachineIRBuilder {
return getMF().getRegInfo().createVirtualRegister(RC);
}
- unsigned getRegFromArg(unsigned Reg) { return Reg; }
+ void addUseFromArg(MachineInstrBuilder &MIB, unsigned Reg) {
+ MIB.addUse(Reg);
+ }
- unsigned getRegFromArg(const MachineInstrBuilder &MIB) {
- return MIB->getOperand(0).getReg();
+ void addUseFromArg(MachineInstrBuilder &MIB, const MachineInstrBuilder &UseMIB) {
+ MIB.addUse(UseMIB->getOperand(0).getReg());
}
+ void addUsesFromArgs(MachineInstrBuilder &MIB) { }
+ template<typename UseArgTy, typename ... UseArgsTy>
+ void addUsesFromArgs(MachineInstrBuilder &MIB, UseArgTy &&Arg1, UseArgsTy &&... Args) {
+ addUseFromArg(MIB, Arg1);
+ addUsesFromArgs(std::forward<UseArgsTy>(Args)...);
+ }
public:
/// Getter for the function we currently build.
MachineFunction &getMF() {
@@ -146,9 +154,7 @@ public:
MachineInstrBuilder buildInstr(unsigned Opc, DstTy &&Ty,
UseArgsTy &&... Args) {
auto MIB = buildInstr(Opc).addDef(getDestFromArg(Ty));
- unsigned It[] = {(getRegFromArg(Args))...};
- for (const auto &i : It)
- MIB.addUse(i);
+ addUsesFromArgs(MIB, std::forward<UseArgsTy>(Args)...);
return MIB;
}
More information about the llvm-commits
mailing list