[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