On 2012.12.11 at 09:13 -0800, Teresa Johnson wrote: > Ping. > Teresa > > On Mon, Nov 19, 2012 at 4:58 PM, Teresa Johnson <tejohn...@google.com> wrote: > > This patch was proposed awhile back, and the new working set program > > summary was split out and committed separately. I would like to > > see if the loop unroller part of the patch can now go in. For the > > previous versions of the patch and related discussion, see: > > > > http://gcc.gnu.org/ml/gcc-patches/2012-06/msg00437.html > > and > > http://gcc.gnu.org/ml/gcc-patches/2012-07/msg01373.html > > > > Use the new working set information in the FDO program summary for > > codesize based unroll and peel decisions, to prevent those > > optimizations from increasing code size too much when the > > program may be sensitive to icache effects. > > > > As part of this, cache additional loop analysis results in the niter_desc > > auxiliary information hanging off the loop structure to reduce > > redundant analyses during unrolling. > > > > Bootstrapped and tested on x86_64-unknown-linux-gnu. Ok for trunk?
I guess this is the missing piece that would fix the Firefox lto/pgo code-size issue: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=45375#c144 -- Markus