]> granicus.if.org Git - llvm/commitdiff
[AArch64][RegisterBankInfo] Rename getValueMappingIdx to getValueMapping
authorQuentin Colombet <qcolombet@apple.com>
Fri, 30 Sep 2016 21:46:19 +0000 (21:46 +0000)
committerQuentin Colombet <qcolombet@apple.com>
Fri, 30 Sep 2016 21:46:19 +0000 (21:46 +0000)
We don't return index, we return the actual ValueMapping.

NFC.

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

lib/Target/AArch64/AArch64GenRegisterBankInfo.def
lib/Target/AArch64/AArch64RegisterBankInfo.cpp

index bbff85f906ea3ea225478f88427aacf2c2560b85..88dac21bab34da9ef1ec5ab453414489d76f59a4 100644 (file)
@@ -101,7 +101,7 @@ RegisterBankInfo::ValueMapping ValMappings[] {
 ///
 /// \pre \p RBIdx != PartialMappingIdx::None
 const RegisterBankInfo::ValueMapping *
-getValueMappingIdx(PartialMappingIdx RBIdx, unsigned Size) {
+getValueMapping(PartialMappingIdx RBIdx, unsigned Size) {
   assert(RBIdx != PartialMappingIdx::None && "No mapping needed for that");
   unsigned ValMappingIdx = First3OpsIdx +
                       (RBIdx + getRegBankBaseIdxOffset(Size)) *
index 35b229a6a0bd3776b24e50c74700b7989c7477fb..3efa0dc53aa4e6e9a4a3d45aeb91de583ead40fd 100644 (file)
@@ -246,12 +246,12 @@ AArch64RegisterBankInfo::getInstrAlternativeMappings(
     if (MI.getNumOperands() != 3)
       break;
     InstructionMappings AltMappings;
-    InstructionMapping GPRMapping(/*ID*/ 1, /*Cost*/ 1,
-                                  getValueMappingIdx(AArch64::FirstGPR, Size),
-                                  /*NumOperands*/ 3);
-    InstructionMapping FPRMapping(/*ID*/ 2, /*Cost*/ 1,
-                                  getValueMappingIdx(AArch64::FirstFPR, Size),
-                                  /*NumOperands*/ 3);
+    InstructionMapping GPRMapping(
+        /*ID*/ 1, /*Cost*/ 1, AArch64::getValueMapping(AArch64::FirstGPR, Size),
+        /*NumOperands*/ 3);
+    InstructionMapping FPRMapping(
+        /*ID*/ 2, /*Cost*/ 1, AArch64::getValueMapping(AArch64::FirstFPR, Size),
+        /*NumOperands*/ 3);
 
     AltMappings.emplace_back(std::move(GPRMapping));
     AltMappings.emplace_back(std::move(FPRMapping));
@@ -352,7 +352,8 @@ AArch64RegisterBankInfo::getInstrMapping(const MachineInstr &MI) const {
         IsFPR ? AArch64::FirstFPR : AArch64::FirstGPR;
 
     return InstructionMapping{DefaultMappingID, 1,
-                              getValueMappingIdx(RBIdx, Size), NumOperands};
+                              AArch64::getValueMapping(RBIdx, Size),
+                              NumOperands};
   }
   default:
     break;
@@ -405,7 +406,8 @@ AArch64RegisterBankInfo::getInstrMapping(const MachineInstr &MI) const {
   SmallVector<const ValueMapping *, 8> OpdsMapping(NumOperands);
   for (unsigned Idx = 0; Idx < NumOperands; ++Idx)
     if (MI.getOperand(Idx).isReg())
-      OpdsMapping[Idx] = getValueMappingIdx(OpRegBankIdx[Idx], OpSize[Idx]);
+      OpdsMapping[Idx] =
+          AArch64::getValueMapping(OpRegBankIdx[Idx], OpSize[Idx]);
 
   Mapping.setOperandsMapping(getOperandsMapping(OpdsMapping));
   return Mapping;