[llvm] [AArch64] Cleanup existing values in getMemOpInfo (PR #98196)

Romain Thomas via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 10 01:39:35 PDT 2024


================
@@ -3731,17 +3954,17 @@ bool AArch64InstrInfo::getMemOpInfo(unsigned Opcode, TypeSize &Scale,
   case AArch64::LDPQpost:
     Scale = TypeSize::getFixed(16);
     Width = TypeSize::getFixed(16);
-    MinOffset = -1024;
-    MaxOffset = 1008;
+    MinOffset = -64;
+    MaxOffset = 63;
     break;
   case AArch64::STPXpre:
   case AArch64::LDPXpost:
   case AArch64::STPDpre:
   case AArch64::LDPDpost:
     Scale = TypeSize::getFixed(8);
     Width = TypeSize::getFixed(8);
----------------
romainthomas wrote:

This shouldn't be `8 * 2` to be consistent with: `LDPXi` cases?

https://github.com/llvm/llvm-project/blob/0f139295aa96c7cf02cbb0de06cea39f51698ae0/llvm/lib/Target/AArch64/AArch64InstrInfo.cpp#L3926

https://github.com/llvm/llvm-project/pull/98196


More information about the llvm-commits mailing list