On 09/17/11 09:16, Alan Modra wrote:
> This patch series adds shrink-wrap support for PowerPC.  The patches
> are on top of Bernd's "Initial shrink-wrapping patch":
> http://gcc.gnu.org/ml/gcc-patches/2011-08/msg02557.html, but with the
> tm.texi patch applied to tm.texi.in.  Bootstrapped and regression
> tested powerpc64-linux all langs except ada, and spec CPU2006 tested.
> The spec results were a little disappointing as I expected to see some
> gains, but my baseline was a -O3 run and I suppose most of the
> shrink-wrap opportunities were lost to inlining.

The last posted version had a bug that crept in during the review cycle,
and which made it quite ineffective. Find this block
          /* As a special case, check for jumps to the last bb that
             cannot successfully be converted to simple_returns later
             on, and mark them as requiring a frame.  These are
             conditional jumps that jump to their fallthru block, so
             it's not a case that is expected to occur often.  */

and replace "condjump_p" in the if statement with "any_condjump_p".

There'll be further patches which increase the number of shrink-wrapping
opportunities we get to see. One of them was submitted earlier:

 http://gcc.gnu.org/ml/gcc-patches/2011-03/msg01499.html
 http://gcc.gnu.org/ml/gcc-patches/2011-03/msg02191.html


Bernd

Reply via email to