[PATCH] D20220: [PEI, AArch64] Use empty spaces in stack area for local stack slot allocation.

Quentin Colombet via llvm-commits llvm-commits at lists.llvm.org
Tue May 31 16:46:29 PDT 2016


qcolombet added a comment.

Hi Geoff,

LGTM.

Did not mark it as accepted as you asked Tim to check the AArch64 part + I am not sure what you'd agreed with Gerolf :).

Cheers,
-Quentin


================
Comment at: lib/CodeGen/PrologEpilogInserter.cpp:599
@@ +598,3 @@
+  for (int i = MinCSFrameIndex; i <= (int)MaxCSFrameIndex; ++i)
+    AllocatedFrameSlots.push_back(i);
+
----------------
No worries, I figured that was what happened :). Phab can be quite confusing after a few revisions.


http://reviews.llvm.org/D20220





More information about the llvm-commits mailing list