[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

Javed Absar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 3 01:49:35 PDT 2018


javed.absar added inline comments.


================
Comment at: lib/Target/AArch64/AArch64MachineFunctionInfo.h:138
 
+  bool usesRedZone() const { return UsesRedZone; }
+  void setUsesRedZone(bool s) { UsesRedZone = s; }
----------------
nitpick - should this be called 'hasRedZone' to be consistent with other namings e.g. hasStackFrame?


https://reviews.llvm.org/D45189





More information about the llvm-commits mailing list