On Fri, Jan 27, 2017 at 05:38:17PM +0100, Jakub Jelinek wrote:
> On Fri, Jan 27, 2017 at 04:34:53PM +0000, Kyrill Tkachov wrote:
> > > +
> > >   <h3 id="powerpc">PowerPC / PowerPC64 / RS6000</h3>
> > >   <ul>
> > >     <li>The PowerPC port now uses LRA by default.</li>
> > >     <li>GCC now diagnoses inline assembly that clobbers register r2.
> > >       This has always been invalid code, and is no longer quietly
> > >       tolerated.</li>
> > > +  <li>Shrink-wrapping optimization can now separate portions of
> > > +    prologues and epilogues to improve performance if some of the
> > > +    work done traditionally by prologues and epilogues is not needed
> > > +    on certain paths.  This is controlled by the
> > > +    <code>-fshrink-wrap-separate</code> option, enabled by default.</li>
> > >   </ul>

Thanks for doing this.  It was still on my todo list :-/

> > AArch64 also implements these hooks and so benefits from the optimisation 
> > as well.
> > Perhaps move this to the general optimizer improvements section and mention 
> > it's only
> > enabled for powerpc and aarch64 for the moment?
> 
> Yeah, I've also noticed that, but not sure what is better, as it is only 2
> targets that support it, so it is not really generic enough.  It is unclear 
> what
> is better.

The subpass _is_ generic, I would move it like Kyrill says.


Segher

Reply via email to