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

Geoff Berry via llvm-commits llvm-commits at lists.llvm.org
Fri May 27 12:28:21 PDT 2016


gberry added a comment.

Hi Gerolf,

I'm not sure I understand exactly what you're asking, but this change will allocate multiple stack slots to free space in the fixed/CS area.  It will assign later slots to this free area even if earlier slots were not assigned there.  It will not assign multiple slots to overlapping regions of the fixed/CS area.


================
Comment at: lib/CodeGen/PrologEpilogInserter.cpp:632
@@ +631,3 @@
+    // clear it to speed up later scavengeStackSlot calls to
+    // StackBytesFree.none()
+    StackBytesFree.clear();
----------------
I suppose, but I don't think it would make the loop any clearer since I don't think it would simplify the control flow of the loop.  Also, this function isn't really doing much else.


http://reviews.llvm.org/D20220





More information about the llvm-commits mailing list