[PATCH] D88983: [NFCI] Replace AArch64StackOffset by StackOffset.

David Sherwood via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 23 07:24:16 PDT 2020


david-arm accepted this revision.
david-arm added a comment.
This revision is now accepted and ready to land.

LGTM!



================
Comment at: llvm/lib/Target/AArch64/AArch64InstrInfo.cpp:3885
   if (IsMulVL)
-    SOffset = StackOffset(Offset, MVT::nxv1i8) +
-              StackOffset(SOffset.getBytes(), MVT::i8);
+    SOffset = StackOffset::getScalable(Offset) +
+              StackOffset::getFixed(SOffset.getFixed());
----------------
nit: You could change the if and else case to also use StackOffset::get().


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D88983/new/

https://reviews.llvm.org/D88983



More information about the llvm-commits mailing list