[PATCH] D45189: [MachineOutliner][AArch64] Keep track of functions that use a red zone in AArch64MachineFunctionInfo and use that instead of checking for noredzone in the MachineOutliner

Jessica Paquette via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 3 13:38:55 PDT 2018


paquette updated this revision to Diff 140853.
paquette edited the summary of this revision.
paquette added a comment.

Updated diff to address comments.

`usesRedZone` is now `hasRedZone`.

`hasRedZone` is now a function that returns an optional. The optional is set either during frame lowering, or on the construction of AArch64FunctionInfo in the case where the IR-level function has the noredzone attribute.

I tried to simplify the test a little, but it needs to be somewhat long to trigger the desired outliner behaviour, unfortunately.


https://reviews.llvm.org/D45189

Files:
  lib/Target/AArch64/AArch64FrameLowering.cpp
  lib/Target/AArch64/AArch64InstrInfo.cpp
  lib/Target/AArch64/AArch64MachineFunctionInfo.h
  test/CodeGen/AArch64/machine-outliner-noredzone.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D45189.140853.patch
Type: text/x-patch
Size: 5054 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180403/d4add76e/attachment.bin>


More information about the llvm-commits mailing list