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]

Re: Now I know why we used to swap builtin_type_void.


On Wed, Dec 06, 2006 at 12:05:11PM -0800, Jim Blandy wrote:
> Why doesn't the same logic apply to the other types created in
> _initialize_gdbtypes?  I believe they're never referred to by debug
> information, and the user has no way to refer to them from the command
> line, so that keeps us out of trouble in those cases.  But if some
> architecture-specific code ever calls lookup_pointer_type on them,
> won't we have the same problem you've discovered above with void?

Exactly.  I think we'd have to move TYPE_POINTER_TYPE out of the type
and into e.g. an architecture hash table to fix this.  Worth it? 
Don't know.

> I'll revert the patch.  It was just a cleanup; I don't actually need
> it for anything.

Thanks.

-- 
Daniel Jacobowitz
CodeSourcery


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