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

2015-05-11 Thread Richard Biener
On Fri, May 8, 2015 at 9:16 PM, Tom de Vries wrote: > 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, th

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

2015-05-08 Thread Tom de Vries
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 form should c

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

2015-05-08 Thread Richard Biener
On Fri, May 8, 2015 at 5:13 PM, Tom de Vries wrote: > Hi, > > this patch fixes PR66013. > > > I. > > Consider this test-case, with a va_list passed from f2 to f2_1: > ... > #include > > inline int __attribute__((always_inline)) > f2_1 (va_list ap) > { > return va_arg (ap, int); > } > > int > f2

[PATCH][PR66013] Update address_taken after ifn_va_arg expansion

2015-05-08 Thread Tom de Vries
Hi, this patch fixes PR66013. I. Consider this test-case, with a va_list passed from f2 to f2_1: ... #include inline int __attribute__((always_inline)) f2_1 (va_list ap) { return va_arg (ap, int); } int f2 (int i, ...) { int res; va_list ap; va_start (ap, i); res = f2_1 (ap); v