[llvm] r281720 - [AArch64][GlobalISel] Use the generic DefaultMapping as the default.

Ahmed Bougacha via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 16 05:33:34 PDT 2016


Author: ab
Date: Fri Sep 16 07:33:34 2016
New Revision: 281720

URL: http://llvm.org/viewvc/llvm-project?rev=281720&view=rev
Log:
[AArch64][GlobalISel] Use the generic DefaultMapping as the default.

This lets generic logic handle the common case, instead of having to
implement applyMappingImpl for each instruction.

Modified:
    llvm/trunk/lib/Target/AArch64/AArch64RegisterBankInfo.cpp

Modified: llvm/trunk/lib/Target/AArch64/AArch64RegisterBankInfo.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/AArch64/AArch64RegisterBankInfo.cpp?rev=281720&r1=281719&r2=281720&view=diff
==============================================================================
--- llvm/trunk/lib/Target/AArch64/AArch64RegisterBankInfo.cpp (original)
+++ llvm/trunk/lib/Target/AArch64/AArch64RegisterBankInfo.cpp Fri Sep 16 07:33:34 2016
@@ -157,7 +157,6 @@ AArch64RegisterBankInfo::getInstrAlterna
 void AArch64RegisterBankInfo::applyMappingImpl(
     const OperandsMapper &OpdMapper) const {
   switch (OpdMapper.getMI().getOpcode()) {
-  case TargetOpcode::G_ADD:
   case TargetOpcode::G_OR: {
     // Those ID must match getInstrAlternativeMappings.
     assert((OpdMapper.getInstrMapping().getID() == 1 ||
@@ -187,7 +186,7 @@ AArch64RegisterBankInfo::getInstrMapping
   else
     BankID = AArch64::GPRRegBankID;
 
-  Mapping = InstructionMapping{1, 1, MI.getNumOperands()};
+  Mapping = InstructionMapping{DefaultMappingID, 1, MI.getNumOperands()};
   int Size = Ty.isValid() ? Ty.getSizeInBits() : 0;
   for (unsigned Idx = 0; Idx < MI.getNumOperands(); ++Idx)
     Mapping.setOperandMapping(Idx, Size, getRegBank(BankID));




More information about the llvm-commits mailing list