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: Thu, 10 Sep 2015 10:51:55 -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> <n99k2ry70lw dot fsf at arm dot com> <20150910154153 dot GE12480 at gate dot crashing dot org>
On Thu, Sep 10, 2015 at 10:41:53AM -0500, Segher Boessenkool wrote:
> > > * shrink-wrap.c (requires_stack_frame_p): Fix formatting.
> > > (dup_block_and_redirect): Delete function.
> > > (can_dup_for_shrink_wrapping): New function.
> > > (fix_fake_fallthrough_edge): New function.
> > > (try_shrink_wrapping): Rewrite function.
> > > (convert_to_simple_return): Call fix_fake_fallthrough_edge.
> >
> > I am interested in the impact on AArch64, but the patch attached can't
> > apply on trunk cleanly after I apply your [1/2], the .rej is quite big.
>
> I rebased it to trunk only yesterday. Huh. I'll have a look.
The patch (taken from the email) applies to trunk just fine for me,
both using git-am and patch.
Segher