This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

[PING^2] [PATCH][5b/5] Avoid running expand_ifn_va_arg_1 in functions without va_arg

From: Tom de Vries <Tom_deVries at mentor dot com>
To: Richard Biener <rguenther at suse dot de>, Jakub Jelinek <jakub at redhat dot com>
Cc: Michael Matz <matz at suse dot de>, GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Thu, 16 Apr 2015 10:50:12 +0200
Subject: [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>

[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 .


Thanks,
- Tom

Follow-Ups:
- Re: [PING^2] [PATCH][5b/5] Avoid running expand_ifn_va_arg_1 in functions without va_arg
  - From: Richard Biener

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]