[rfc] [2/2] Remove macros TARGET_{FLOAT,DOUBLE,LONG_DOUBLE}_{BIT|FORMAT}

Ulrich Weigand uweigand@de.ibm.com
Tue Jun 12 16:37:00 GMT 2007


Markus Deuling wrote:

> Sorry. Attached is a rebased version that applies against current head :-)

Thanks.  One more thing: I've found an inconsistency in the gdbarch.c /
gdbarch.sh changes:

> diff -urN src/gdb/gdbarch.c dev/gdb/gdbarch.c
> --- src/gdb/gdbarch.c	2007-06-12 18:05:43.000000000 +0200
> +++ dev/gdb/gdbarch.c	2007-06-12 18:07:23.000000000 +0200
> @@ -373,7 +373,7 @@
>                 struct gdbarch_tdep *tdep)
>  {
>    /* NOTE: The new architecture variable is named ``current_gdbarch''
> -     so that macros such as TARGET_DOUBLE_BIT, when expanded, refer to
> +     so that macros such as TARGET_ARCHITECTURE, when expanded, refer to
>       the current local architecture and not the previous global
>       architecture.  This ensures that the new architectures initial
>       values are not influenced by the previous architecture.  Once

> diff -urN src/gdb/gdbarch.sh dev/gdb/gdbarch.sh
> --- src/gdb/gdbarch.sh	2007-06-12 18:05:43.000000000 +0200
> +++ dev/gdb/gdbarch.sh	2007-06-12 18:07:23.000000000 +0200
> @@ -1348,7 +1348,7 @@
>                 struct gdbarch_tdep *tdep)
>  {
>    /* NOTE: The new architecture variable is named \`\`current_gdbarch''
> -     so that macros such as TARGET_DOUBLE_BIT, when expanded, refer to
> +     so that macros such as gdbarch_double_bit, when expanded, refer to
>       the current local architecture and not the previous global
>       architecture.  This ensures that the new architectures initial
>       values are not influenced by the previous architecture.  Once

I've now committed the patch with this fixed.

Bye,
Ulrich

-- 
  Dr. Ulrich Weigand
  GNU Toolchain for Linux on System z and Cell BE
  Ulrich.Weigand@de.ibm.com



More information about the Gdb-patches mailing list