[PATCH] D27336: [globalisel][aarch64] Replace magic numbers with corresponding enumerators in ValMappings. NFC

[Daniel Sanders via Phabricator via llvm-commits]

dsanders created this revision.
dsanders added reviewers: ab, qcolombet, t.p.northover.
dsanders added a subscriber: llvm-commits.
Herald added subscribers: rovka, dberris, vkalintiris, rengolin, aemerson.

https://reviews.llvm.org/D27336

Files:
  lib/Target/AArch64/AArch64GenRegisterBankInfo.def


Index: lib/Target/AArch64/AArch64GenRegisterBankInfo.def
===================================================================
--- lib/Target/AArch64/AArch64GenRegisterBankInfo.def
+++ lib/Target/AArch64/AArch64GenRegisterBankInfo.def
@@ -37,7 +37,8 @@
   FirstGPR = GPR32,
   LastGPR = GPR64,
   FirstFPR = FPR32,
-  LastFPR = FPR512
+  LastFPR = FPR512,
+  PartialMappingIdx_Min = FirstGPR,
 };
 
 static unsigned getRegBankBaseIdxOffset(unsigned Size) {
@@ -81,28 +82,28 @@
   // 3-operands instructions (all binary operations should end up with one of
   // those mapping).
   // 0: GPR 32-bit value. <-- This must match First3OpsIdx.
-  {&PartMappings[0], 1}, {&PartMappings[0], 1}, {&PartMappings[0], 1},
+  {&PartMappings[GPR32 - PartialMappingIdx_Min], 1}, {&PartMappings[GPR32 - PartialMappingIdx_Min], 1}, {&PartMappings[GPR32 - PartialMappingIdx_Min], 1},
   // 3: GPR 64-bit value.
-  {&PartMappings[1], 1}, {&PartMappings[1], 1}, {&PartMappings[1], 1},
+  {&PartMappings[GPR64 - PartialMappingIdx_Min], 1}, {&PartMappings[GPR64 - PartialMappingIdx_Min], 1}, {&PartMappings[GPR64 - PartialMappingIdx_Min], 1},
   // 6: FPR 32-bit value.
-  {&PartMappings[2], 1}, {&PartMappings[2], 1}, {&PartMappings[2], 1},
+  {&PartMappings[FPR32 - PartialMappingIdx_Min], 1}, {&PartMappings[FPR32 - PartialMappingIdx_Min], 1}, {&PartMappings[FPR32 - PartialMappingIdx_Min], 1},
   // 9: FPR 64-bit value.
-  {&PartMappings[3], 1}, {&PartMappings[3], 1}, {&PartMappings[3], 1},
+  {&PartMappings[FPR64 - PartialMappingIdx_Min], 1}, {&PartMappings[FPR64 - PartialMappingIdx_Min], 1}, {&PartMappings[FPR64 - PartialMappingIdx_Min], 1},
   // 12: FPR 128-bit value.
-  {&PartMappings[4], 1}, {&PartMappings[4], 1}, {&PartMappings[4], 1},
+  {&PartMappings[FPR128 - PartialMappingIdx_Min], 1}, {&PartMappings[FPR128 - PartialMappingIdx_Min], 1}, {&PartMappings[FPR128 - PartialMappingIdx_Min], 1},
   // 15: FPR 256-bit value.
-  {&PartMappings[5], 1}, {&PartMappings[5], 1}, {&PartMappings[5], 1},
+  {&PartMappings[FPR256 - PartialMappingIdx_Min], 1}, {&PartMappings[FPR256 - PartialMappingIdx_Min], 1}, {&PartMappings[FPR256 - PartialMappingIdx_Min], 1},
   // 18: FPR 512-bit value. <-- This must match Last3OpsIdx.
-  {&PartMappings[6], 1}, {&PartMappings[6], 1}, {&PartMappings[6], 1},
+  {&PartMappings[FPR512 - PartialMappingIdx_Min], 1}, {&PartMappings[FPR512 - PartialMappingIdx_Min], 1}, {&PartMappings[FPR512 - PartialMappingIdx_Min], 1},
   // Cross register bank copies.
   // 21: GPR 32-bit value to FPR 32-bit value. <-- This must match FirstCrossRegCpyIdx.
-  {&PartMappings[0], 1}, {&PartMappings[2], 1},
+  {&PartMappings[GPR32 - PartialMappingIdx_Min], 1}, {&PartMappings[FPR32 - PartialMappingIdx_Min], 1},
   // 23: GPR 64-bit value to FPR 64-bit value.
-  {&PartMappings[1], 1}, {&PartMappings[3], 1},
+  {&PartMappings[GPR64 - PartialMappingIdx_Min], 1}, {&PartMappings[FPR64 - PartialMappingIdx_Min], 1},
   // 25: FPR 32-bit value to GPR 32-bit value.
-  {&PartMappings[2], 1}, {&PartMappings[0], 1},
+  {&PartMappings[FPR32 - PartialMappingIdx_Min], 1}, {&PartMappings[GPR32 - PartialMappingIdx_Min], 1},
   // 27: FPR 64-bit value to GPR 64-bit value. <-- This must match LastCrossRegCpyIdx.
-  {&PartMappings[3], 1}, {&PartMappings[1], 1}
+  {&PartMappings[FPR64 - PartialMappingIdx_Min], 1}, {&PartMappings[GPR64 - PartialMappingIdx_Min], 1}
 
 };
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D27336.80052.patch
Type: text/x-patch
Size: 3372 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161202/d4e3e17e/attachment.bin>