[PATCH] D59713: [ARM] Add missing memory operands to a bunch of instructions.

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 23 02:50:14 PDT 2019


SjoerdMeijer accepted this revision.
SjoerdMeijer added a comment.
This revision is now accepted and ready to land.

LGTM, just a question inline.



================
Comment at: lib/Target/ARM/ARMBaseInstrInfo.cpp:1180
+  if (MI.mayStore() && hasStoreToStackSlot(MI, Accesses) &&
+      Accesses.size() == 1) {
     FrameIndex =
----------------
I can see that this extra check makes sense, just curious how it relates to the other changes.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59713





More information about the llvm-commits mailing list