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]

Re: [PATCH][PR66013] Update address_taken after ifn_va_arg expansion

From: Tom de Vries <Tom_deVries at mentor dot com>
To: Richard Biener <richard dot guenther at gmail dot com>
Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Fri, 8 May 2015 21:16:48 +0200
Subject: Re: [PATCH][PR66013] Update address_taken after ifn_va_arg expansion
Authentication-results: sourceware.org; auth=none
References: <554CD2AE dot 7010304 at mentor dot com> <CAFiYyc1xy30_=TNSemYu7Etp07UQibhdoF5bB5fD5Fd++ma=Qw at mail dot gmail dot com>

On 08-05-15 17:31, Richard Biener wrote:

>OK for trunk?

As noted in one of the PRs I think that it is the proper time to
re-implement the stdarg optimization on the un-lowered form which
should also fix this.

AFAIU, the implementation of the stdarg optimization on the un-lowered formshould contain an independent fix for this problem, and this fix won't befunctional any more then, so indeed, it's a patch with (hopefully) a limitedlifespan.

But I thought that the patch is simple and low-risk enough to accept for trunknow, hence the submission.

Also, I suppose it's not bad to start developing from a situation where moretests are passing.


Thanks,
- Tom

Follow-Ups:
- Re: [PATCH][PR66013] Update address_taken after ifn_va_arg expansion
  - From: Richard Biener

References:
- [PATCH][PR66013] Update address_taken after ifn_va_arg expansion
  - From: Tom de Vries
- Re: [PATCH][PR66013] Update address_taken after ifn_va_arg expansion
  - From: Richard Biener

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