From: Quentin Colombet <qcolombet@apple.com>
Date: Fri, 30 Sep 2016 21:46:21 +0000 (+0000)
Subject: [AArch64][RegisterBankInfo] Use the helper functions for the checks
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=e71d314ca79527f810e11cc9ab968b76043706e3;p=llvm

[AArch64][RegisterBankInfo] Use the helper functions for the checks

This makes sure the helper functions work as expected.

NFC.

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

diff --git a/lib/Target/AArch64/AArch64RegisterBankInfo.cpp b/lib/Target/AArch64/AArch64RegisterBankInfo.cpp
index 3efa0dc53aa..fdd4de4deae 100644
--- a/lib/Target/AArch64/AArch64RegisterBankInfo.cpp
+++ b/lib/Target/AArch64/AArch64RegisterBankInfo.cpp
@@ -130,48 +130,45 @@ AArch64RegisterBankInfo::AArch64RegisterBankInfo(const TargetRegisterInfo &TRI)
   CHECK_PARTIALMAP(FPR512, 0, 512, RBFPR);
 
 // Check value mapping.
-#define CHECK_VALUEMAP_IMPL(ValIdx, PartIdx)                                   \
+#define CHECK_VALUEMAP_IMPL(RBName, Size, Offset)                              \
   do {                                                                         \
-    unsigned PartialMapBaseIdx = AArch64::PartialMappingIdx::PartIdx;          \
+    AArch64::PartialMappingIdx PartialMapBaseIdx =                             \
+        AArch64::PartialMappingIdx::RBName##Size;                              \
     (void) PartialMapBaseIdx;                                                  \
-    const ValueMapping &Map = AArch64::ValMappings[ValIdx];                    \
+    const ValueMapping &Map =                                                  \
+        AArch64::getValueMapping(AArch64::First##RBName, Size)[Offset];        \
     (void) Map;                                                                \
     assert(Map.BreakDown == &AArch64::PartMappings[PartialMapBaseIdx] &&       \
-           Map.NumBreakDowns == 1 && #ValIdx " " #PartIdx                      \
-                                             " is incorrectly initialized");   \
+           Map.NumBreakDowns == 1 && #RBName #Size                             \
+           " " #Offset " is incorrectly initialized");                         \
   } while (0)
 
-#define CHECK_VALUEMAP(Idx)                                                    \
-  CHECK_VALUEMAP_IMPL((AArch64::PartialMappingIdx::Idx *                       \
-                       AArch64::ValueMappingIdx::DistanceBetweenRegBanks),     \
-                      Idx)
+#define CHECK_VALUEMAP(RBName, Size) CHECK_VALUEMAP_IMPL(RBName, Size, 0)
 
-  CHECK_VALUEMAP(GPR32);
-  CHECK_VALUEMAP(GPR64);
-  CHECK_VALUEMAP(FPR32);
-  CHECK_VALUEMAP(FPR64);
-  CHECK_VALUEMAP(FPR128);
-  CHECK_VALUEMAP(FPR256);
-  CHECK_VALUEMAP(FPR512);
+  CHECK_VALUEMAP(GPR, 32);
+  CHECK_VALUEMAP(GPR, 64);
+  CHECK_VALUEMAP(FPR, 32);
+  CHECK_VALUEMAP(FPR, 64);
+  CHECK_VALUEMAP(FPR, 128);
+  CHECK_VALUEMAP(FPR, 256);
+  CHECK_VALUEMAP(FPR, 512);
 
 // Check the value mapping for 3-operands instructions where all the operands
 // map to the same value mapping.
-#define CHECK_VALUEMAP_3OPS(Idx)                                               \
+#define CHECK_VALUEMAP_3OPS(RBName, Size)                                      \
   do {                                                                         \
-    unsigned BaseIdx =                                                         \
-        AArch64::First3OpsIdx + AArch64::PartialMappingIdx::Idx * 3;           \
-    CHECK_VALUEMAP_IMPL(BaseIdx, Idx);                                         \
-    CHECK_VALUEMAP_IMPL(BaseIdx + 1, Idx);                                     \
-    CHECK_VALUEMAP_IMPL(BaseIdx + 2, Idx);                                     \
+    CHECK_VALUEMAP_IMPL(RBName, Size, 0);                                      \
+    CHECK_VALUEMAP_IMPL(RBName, Size, 1);                                      \
+    CHECK_VALUEMAP_IMPL(RBName, Size, 2);                                      \
   } while (0)
 
-  CHECK_VALUEMAP_3OPS(GPR32);
-  CHECK_VALUEMAP_3OPS(GPR64);
-  CHECK_VALUEMAP_3OPS(FPR32);
-  CHECK_VALUEMAP_3OPS(FPR64);
-  CHECK_VALUEMAP_3OPS(FPR128);
-  CHECK_VALUEMAP_3OPS(FPR256);
-  CHECK_VALUEMAP_3OPS(FPR512);
+  CHECK_VALUEMAP_3OPS(GPR, 32);
+  CHECK_VALUEMAP_3OPS(GPR, 64);
+  CHECK_VALUEMAP_3OPS(FPR, 32);
+  CHECK_VALUEMAP_3OPS(FPR, 64);
+  CHECK_VALUEMAP_3OPS(FPR, 128);
+  CHECK_VALUEMAP_3OPS(FPR, 256);
+  CHECK_VALUEMAP_3OPS(FPR, 512);
 
   assert(verify(TRI) && "Invalid register bank information");
 }