[PATCH][PR66010] Don't take address of ap unless necessary

[Michael Matz]

Hi,

On Fri, 8 May 2015, Tom de Vries wrote:

> III.
> 
> Using the patch, before inlining we can see the address operator has been
> removed in va_arg:
> ...
> f2_1 (struct  * apD.1832)
> {
>   intD.6 _4;
> 
>   # .MEM_3 = VDEF <.MEM_1(D)>
>   # USE = anything
>   # CLB = anything
> 
>   _4 = VA_ARG (ap_2(D), 0B, 0);
>   # VUSE <.MEM_3>
>   return _4;
> }

Have you visually inspected that the address operator stays on 
architectures where va_list is e.g. a char pointer (i.e. on those where 
VA_ARG has the side-effect of modifying ap)?


Ciao,
Michael.