Re: gimplify.c fix for rs6000 and mips

[Andrew MacLeod <amacleod at redhat dot com>]

On 10/10/2013 10:35 AM, Richard Biener wrote:
> On Thu, Oct 10, 2013 at 4:30 PM, Richard Biener
> <richard.guenther@gmail.com> wrote:
> > On Thu, Oct 10, 2013 at 3:10 PM, Andrew MacLeod <amacleod@redhat.com> wrote:
> > > More fun target specific stuff.  gimplify.c needs the back end supplied
> > > macros/function for the va-arg padding functions to bootstrap...
> > Err ... don't we have the va_arg gimplify target hooks to hide this
> > kind of target dependency?
> Ah - it's common code for the targets...  which makes me believe
> it belongs in targhooks.c, kind of.
>
> Btw, you didn't move gimplify_va_arg_expr to gimplify.c for some reason?
yeah, gimplify.c already was calling it, so it seemed more appropriate 
for a gimplification routine in gimplify.c.  If we didn't move it, 
gimpliify.c would need builtins.h for no reason other than to get the 
prototype (once it was moved out of tree.h where it happened to be 
declared)  I didn't realize it would drag crud with it :-P

seems like there should be a better solution for PAD_VARARGS_DOWN 
(defined in config/targ.h) and ARGS_GROW_DOWNWARD.. and somewhere in 
there something gets expanded to ARGS_SIZE_RTX which required expr.h  
:-P  . Its a bit ugly for sure.

At least it compiles now.  I wont get to anything until next week as Im 
almost out for the rest of the week now, but then I'll look to see if it 
can be cleaned up  a bit.

Andrew