[PATCH] D15393: [X86] Order the local stack symbols to improve code size and locality.
Zia Ansari via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 22 10:00:14 PST 2016
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.
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.
More information about the llvm-commits