This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB 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]

[commit] Avoid builtin_type_int during type construction


I wanted to construct a new type during gdbarch initialization and
ran into two problems caused by the use of builtin_type_int, which
is now a macro referencing current_gdbarch.

Every range type created uses builtin_type_int as the type of its
fields (which hold the range's endpoints).  There's an unclear
FIXME here.  The use of int isn't necessary; nothing ever looks at the
types of these fields, and the comments in gdbtypes.h suggest that
their types are unused.

And every vector type is an array using int as its index type.  Since
vectors are small arrays of constant size, there's no worry about this
type; int32 works just as well.

Tested on x86_64-linux and committed.

-- 
Daniel Jacobowitz
CodeSourcery

2007-10-03  Daniel Jacobowitz  <dan@codesourcery.com>

	* gdbtypes.c (create_range_type): Do not set TYPE_FIELD_TYPE for the
	bounds.
	(init_vector_type): Use builtin_type_int32.

Index: gdbtypes.c
===================================================================
RCS file: /cvs/src/src/gdb/gdbtypes.c,v
retrieving revision 1.134
diff -u -p -r1.134 gdbtypes.c
--- gdbtypes.c	5 Sep 2007 00:51:48 -0000	1.134
+++ gdbtypes.c	3 Oct 2007 20:36:41 -0000
@@ -705,8 +705,6 @@ create_range_type (struct type *result_t
   memset (TYPE_FIELDS (result_type), 0, 2 * sizeof (struct field));
   TYPE_FIELD_BITPOS (result_type, 0) = low_bound;
   TYPE_FIELD_BITPOS (result_type, 1) = high_bound;
-  TYPE_FIELD_TYPE (result_type, 0) = builtin_type_int;	/* FIXME */
-  TYPE_FIELD_TYPE (result_type, 1) = builtin_type_int;	/* FIXME */
 
   if (low_bound >= 0)
     TYPE_FLAGS (result_type) |= TYPE_FLAG_UNSIGNED;
@@ -950,7 +948,7 @@ init_vector_type (struct type *elt_type,
  
   array_type = create_array_type (0, elt_type,
 				  create_range_type (0, 
-						     builtin_type_int,
+						     builtin_type_int32,
 						     0, n-1));
   make_vector_type (array_type);
   return array_type;


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