This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH 2/2] shrink-wrap: Rewrite try_shrink_wrapping
- From: Segher Boessenkool <segher at kernel dot crashing dot org>
- To: Jiong Wang <jiong dot wang at arm dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Fri, 11 Sep 2015 07:08:06 -0500
- Subject: Re: [PATCH 2/2] shrink-wrap: Rewrite try_shrink_wrapping
- Authentication-results: sourceware.org; auth=none
- References: <5503d4c7cce8849f8a2901b540b388411b6b6a5a dot 1441893071 dot git dot segher at kernel dot crashing dot org> <c66f1521d7ec462efb6683f4b44beecaf7e3553f dot 1441893071 dot git dot segher at kernel dot crashing dot org> <20150911021715 dot GJ12480 at gate dot crashing dot org> <n99y4gdtihu dot fsf at arm dot com>
On Fri, Sep 11, 2015 at 10:24:42AM +0100, Jiong Wang wrote:
>
> Segher Boessenkool writes:
>
> > On Thu, Sep 10, 2015 at 08:14:29AM -0700, Segher Boessenkool wrote:
> >> This patch rewrites the shrink-wrapping algorithm, allowing non-linear
> >> pieces of CFG to be duplicated for use without prologue instead of just
> >> linear pieces.
> >
> >> Bootstrapped and regression tested on powerpc64-linux. Is this okay
> >> for mainline?
> >
> > Now also bootstrapped and regression tested on x86_64-linux.
>
> + AArch64 boostrapping OK.
Thank you for testing!
> A quick check shows > 30% more functions shrink-wrapped during
> bootstrapping by a the following command:
>
> cd $TOP_BUILD ; find . -name "*.pro_and_epilogue" | xargs grep "Perform.*shrink" | wc -l
Wow, that is a lot! But this is mostly the testsuite? Shorter functions
can be wrapped a whole lot more often.
Segher