This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[rfc][18/37] Eliminate builtin_type_ macros: Ada System.Address special handling
- From: uweigand at de dot ibm dot com
- To: gdb-patches at sourceware dot org
- Date: Sun, 31 Aug 2008 19:51:03 +0200
- Subject: [rfc][18/37] Eliminate builtin_type_ macros: Ada System.Address special handling
- References: <20080831175045.128504000@de.ibm.com>
Hello,
this eliminates a hard-coded use of builtin_type_void_data_ptr in
ada_val_print_1.
This use occurs in code to work around some GNAT versions that encode
System.Address as an integer instead of pointer type. The following
patch uses the gdbarch associates with the objfile where the type was
defined to determine the pointer type to use for this work around.
This is possible since for all types without objfile (i.e. builtin
types), the work-around is not required as GDB installs them as
pointer types to start with.
Bye,
Ulrich
ChangeLog:
* ada-valprint.c: Include "objfiles.h".
(ada_val_print_1): Use the gdbarch associated with the objfile whether
a System.Address type is defined to retrieve the proper pointer type
to use to print it.
Index: gdb-head/gdb/ada-valprint.c
===================================================================
--- gdb-head.orig/gdb/ada-valprint.c
+++ gdb-head/gdb/ada-valprint.c
@@ -33,6 +33,7 @@
#include "c-lang.h"
#include "infcall.h"
#include "exceptions.h"
+#include "objfiles.h"
/* Encapsulates arguments to ada_val_print. */
struct ada_val_print_args
@@ -792,18 +793,27 @@ ada_val_print_1 (struct type *type, cons
{
print_scalar_formatted (valaddr, type, format, 0, stream);
}
- else if (ada_is_system_address_type (type))
+ else if (ada_is_system_address_type (type)
+ && TYPE_OBJFILE (type) != NULL)
{
/* FIXME: We want to print System.Address variables using
the same format as for any access type. But for some
reason GNAT encodes the System.Address type as an int,
so we have to work-around this deficiency by handling
- System.Address values as a special case. */
+ System.Address values as a special case.
+
+ We do this only for System.Address types defined in an
+ objfile. For the built-in version of System.Address we
+ have installed the proper type to begin with. */
+
+ struct gdbarch *gdbarch = get_objfile_arch (TYPE_OBJFILE (type));
+ struct type *ptr_type = builtin_type (gdbarch)->builtin_data_ptr;
+
fprintf_filtered (stream, "(");
type_print (type, "", stream, -1);
fprintf_filtered (stream, ") ");
fputs_filtered (paddress (extract_typed_address
- (valaddr, builtin_type_void_data_ptr)),
+ (valaddr, ptr_type)),
stream);
}
else
--
Dr. Ulrich Weigand
GNU Toolchain for Linux on System z and Cell BE
Ulrich.Weigand@de.ibm.com