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: new argument macro: INITIAL_ARGS_SIZE


On Fri, Mar 01, 2002 at 11:29:54AM +1100, Aldy Hernandez wrote:
> STARTING_FRAME_OFFSSET does not include INITIAL_ARGS_SIZE currently:
> 
> #define STARTING_FRAME_OFFSET                                           \
>    (RS6000_ALIGN (current_function_outgoing_args_size,                   \
>                   TARGET_ALTIVEC ? 16 : 8)                               \
>     + RS6000_VARARGS_AREA                                                \
>     + RS6000_SAVE_AREA)

Yes it does.  You can't see it because it's hidden inside
current_function_outgoing_args_size.

Recall that we set args_size.constant to INITIAL_ARGS_SIZE.
Well, current_function_outgoing_args_size is merely the max
value that args_size.constant ever has for any function call.

> > Also, STARTING_FRAME_OFFSET should be using STACK_POINTER_OFFSET
> > instead of RS6000_SAVE_AREA.  As it is you're counting that space
> 
> STACK_POINTER_OFFSSET is 0, remember.

Yes, I remember.  But pedantically, that's the correct thing to do.


r~


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