[PATCH] D80125: [ARM][MachineOutliner] Add NoLRSave mode.

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 9 04:21:20 PDT 2020


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

Thanks, LGTM.



================
Comment at: llvm/lib/Target/ARM/ARMBaseInstrInfo.cpp:5700
+    unsigned NumBytesNoStackCalls = 0;
+    std::vector<outliner::Candidate> CandidatesWithoutStackFixups;
+
----------------
yroux wrote:
> samparker wrote:
> > SmallVector?
> It will be passed as a parameter to outliner::OutlinedFunction which waits for an std::Vector, maybe a SmallVector can used there instead but I think it should be done in another patch
Okay, fair enough.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80125





More information about the llvm-commits mailing list