On Mon, May 06, 2019 at 09:55:50PM +0000, Segher Boessenkool wrote:
> We don't need this.
>
>
> Segher
>
>
> 2019-05-06 Segher Boessenkool <seg...@kernel.crashing.org>
>
> * config/rs6000/rs6000.h (PRE_GCC3_DWARF_FRAME_REGISTERS): Delete.
Why do you think so?
This seems to be a clear ABI break to me in the __frame_state_for
API.
So, if a __frame_state_for caller calls the function, it will overflow
the buffer passed by the caller.
> diff --git a/gcc/config/rs6000/rs6000.h b/gcc/config/rs6000/rs6000.h
> index ff9449c..3829e8f 100644
> --- a/gcc/config/rs6000/rs6000.h
> +++ b/gcc/config/rs6000/rs6000.h
> @@ -817,9 +817,6 @@ enum data_align { align_abi, align_opt, align_both };
>
> #define FIRST_PSEUDO_REGISTER 115
>
> -/* This must be included for pre gcc 3.0 glibc compatibility. */
> -#define PRE_GCC3_DWARF_FRAME_REGISTERS 77
> -
> /* The sfp register and 3 HTM registers
> aren't included in DWARF_FRAME_REGISTERS. */
> #define DWARF_FRAME_REGISTERS (FIRST_PSEUDO_REGISTER - 4)
> --
> 1.8.3.1
Jakub
