This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PING^2] [PATCH][5b/5] Avoid running expand_ifn_va_arg_1 in functions without va_arg
- From: Richard Biener <rguenther at suse dot de>
- To: Tom de Vries <Tom_deVries at mentor dot com>
- Cc: Jakub Jelinek <jakub at redhat dot com>, Michael Matz <matz at suse dot de>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 16 Apr 2015 10:56:29 +0200 (CEST)
- Subject: Re: [PING^2] [PATCH][5b/5] Avoid running expand_ifn_va_arg_1 in functions without va_arg
- Authentication-results: sourceware.org; auth=none
- References: <54E5BB06 dot 2080102 at mentor dot com> <54E5D0D2 dot 8080306 at mentor dot com> <alpine dot LSU dot 2 dot 11 dot 1502191400500 dot 28824 at zhemvz dot fhfr dot qr> <54E9D5E2 dot 3070805 at mentor dot com> <alpine dot LNX dot 2 dot 00 dot 1502230909590 dot 18611 at wotan dot suse dot de> <54EAFC5B dot 1030003 at mentor dot com> <54EC1ECB dot 5030609 at mentor dot com> <54FF0E47 dot 2070200 at mentor dot com> <552F77C4 dot 1000706 at mentor dot com>
On Thu, 16 Apr 2015, Tom de Vries wrote:
> [stage1 ping^2]
> On 10-03-15 16:31, Tom de Vries wrote:
> > [stage1 ping]
> > [was: Postpone expanding va_arg until pass_stdarg]
> > On 24-02-15 07:48, Tom de Vries wrote:
> > > On 23-02-15 11:09, Tom de Vries wrote:
> > > > On 23-02-15 09:26, Michael Matz wrote:
> > > > > Hi,
> > > > >
> > > > > On Sun, 22 Feb 2015, Tom de Vries wrote:
> > > > >
> > > > > > Btw, I'm wondering if as run-time optimization we can tentatively
> > > > > > set
> > > > > > PROP_gimple_lva at the start of the gimple pass, and unset it in
> > > > > > gimplify_va_arg_expr. That way we would avoid the loop in
> > > > > > expand_ifn_va_arg_1 (over all bbs and gimples) in functions without
> > > > > > va_arg.
> > > > >
> > > > > That makes sense.
> > > > >
> > > >
> > > > I'll put this follow-up patch through testing.
> > >
> > > Bootstrapped and reg-tested as before together with the rest of the patch
> > > series.
> > >
> > > OK for stage1?
> > >
> >
> > Ping.
> >
>
> Ping again.
>
> Patch originally posted at:
> https://gcc.gnu.org/ml/gcc-patches/2015-02/msg01402.html .
Ok.
Thanks,
Richard.
> Thanks,
> - Tom
>
>
>
--
Richard Biener <rguenther@suse.de>
SUSE LINUX GmbH, GF: Felix Imendoerffer, Jane Smithard, Jennifer Guild,
Dilip Upmanyu, Graham Norton HRB 21284 (AG Nuernberg)