John David Anglin <dave.ang...@bell.net> writes: > On 2018-01-16 5:52 AM, Richard Sandiford wrote: >> 2018-01-16 Richard Sandiford<richard.sandif...@linaro.org> >> >> gcc/ >> PR target/83858 >> * config/pa/pa.h (FUNCTION_ARG_SIZE): Delete. >> * config/pa/pa-protos.h (pa_function_arg_size): Declare. >> * config/pa/som.h (ASM_DECLARE_FUNCTION_NAME): Use >> pa_function_arg_size instead of FUNCTION_ARG_SIZE. >> * config/pa/pa.c (pa_function_arg_advance): Likewise. >> (pa_function_arg, pa_arg_partial_bytes): Likewise. >> (pa_function_arg_size): New function. > Thanks Richard. I started a build yesterday evening with essentially > the same change. > > Two little nits. I believe a declaration for pa_function_arg_size needs > to be added to pa-protos.h.
The patch did have this. > Secondly, the comment for pa_function_arg_size needs to be updated to > say "function" instead of "macro". Otherwise, the change is okay. Oops, yes. Installed with that change, thanks. Richard