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

[Zia Ansari via llvm-commits]

zansari added a comment.

In http://reviews.llvm.org/D15393#355355, @Gerolf wrote:

> Hi Zia,
>
> do you have HSW performance numbers for this change? An internal bot is logging a >10% regression for MultiSource/Benchmarks/Ptrdist/ks/ks https://smooshbase.apple.com/perf/db_default/v4/nts/graph?highlight_run=114068&plot.746=313.746.3 (O3 flto) pinned to this change.
>
> Thanks!
> Gerolf


Hi Gerolf,

I ran internal performance testing on Atom and HSW (spec2k/2006, eembc, and a few other benchmarks), and I didn't see any negative swings. I'll be happy to look into your regression if you can give me something to look at. The link doesn't open for me (page not available).

> If you can reproduce it perhaps there is a tuning opportunity.


Is this a test to which I can get access easily? Let me know how I can get/run it and I'll take a look.

Thanks,
Zia.


Repository:
  rL LLVM

http://reviews.llvm.org/D15393