Hi Honza,

thanks for putting this together.  Please find some notes below.
Can you please take these plus the other bits of feedback you got
and post an updated patch for a quick final check?

On Wednesday 2015-01-21 22:58, Jan Hubicka wrote:
> Index: changes.html
> ===================================================================
> +     <li>New identical code folding pass (controled via 
> <code>-fipa-icf</code>).

"An Identical Code Folding (ICF) pass...has been added."

> +      Compared to the identical code folding performed by Gold linker this

"by the Gold linker"

> +      pass does not require function sections.  It also performs merging
> +      pre-inline so inter-procedural optimizations are aware of the

pre-inlining or before inlining?

> +      code re-use. On the other hand not all unifications performed
> +      by linker are doable by GCC pass; the GCC pass must honnor

"by a linker"

"doable by GCC which must honor" (no double n in honor)

> +      aliasing information. During link-time optimization of Firefox,
> +      the pass unify about 29000 functions that is 10% overall</li>

"this pass unifies"

"that is..." -> "or 10% overall." (with full stop)

> +     <li>Devirtualization pass was significantly improved by adding

"The Devirtualization pass..."

> +      <li>New auto-FDO mode that use profiles collected by low overhead
> +       profiling tools (perf) instead of more expensive program

"A new auto-FDO mode uses ..."

> +       instrumentation (via <code>-fprofile-generate</code>). SPEC2006
> +       benchmarks improve by 4.7% with auto-FDO and by 7.3% with
> +       traditional feedback directed optimization.

On which architecture?

Gerald

Reply via email to