[PATCH] D15393: [X86] Order the local stack symbols to improve code size and locality.

[Sanjay Patel via llvm-commits]

spatel added a subscriber: spatel.
spatel added a comment.

In http://reviews.llvm.org/D15393#347116, @silvas wrote:

> One interesting thing to also note: the 11, 9, and 12 rows have such bloated instruction sizes primarily because of needing to store an immediate "0" as an imm32. (we have some open PR's about this I think).


Excellent data visualization - thanks! If you have a chance, it would be interesting to recompile the app that you're looking at with this patch applied to see if that app benefits.

The related instruction-size bugzillas that I filed are:
https://llvm.org/bugs/show_bug.cgi?id=24447
https://llvm.org/bugs/show_bug.cgi?id=24448
https://llvm.org/bugs/show_bug.cgi?id=24449

We made some progress on the example in the last report, but that may not apply to the cases that you were looking at. It's possible that this helps too:
http://reviews.llvm.org/rL256969


http://reviews.llvm.org/D15393