[PATCH] D23366: [AArch64] Assign stack locations to increase load/store pairing.

[Chad Rosier via llvm-commits]

mcrosier added a comment.

The code looks reasonable and it appears to do what's advertised.

Did you measure how many more instructions are paired?  I assume this is mostly a code size reduction optimization.  Did you run into any issues with compile-time?


================
Comment at: test/CodeGen/AArch64/stack-layout-pairing.ll:38
@@ +37,3 @@
+; Check that %x1/%y1 and %x1/%x2 get allocated to adjacent stack locations and
+; loaded using load pair instructions when optimizing for size.
+define void @test2(i32 %c) {
----------------
I don't understand this comment about optimizing for size..


https://reviews.llvm.org/D23366