This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC 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: [PATCH] Ensure DW_TAG_base_type has DW_AT_name (PR debug/42278)


On 05/11/2010 03:59 PM, Jakub Jelinek wrote:
I think the TYPE_PRECISION check might be necessary, at least set_sizetype
has some code to change precision from the base type.

The precision code in set_sizetype is for bitsizetype, not sizetype.


And, it seems many frontends have NULL TYPE_NAME on sizetype, or
it is an identifier, not a TYPE_DECL.

Right, I figured the first three lines would stay.


BTW, it seems there is another call to base_type_die in
dwarf2out_imported_module_or_decl_1:
       if (is_base_type (TREE_TYPE (decl)))
         at_import_die = base_type_die (TREE_TYPE (decl));

Well, that seems clearly wrong. We shouldn't generate an additional DIE for 'int' just because we have a using-declaration for an int typedef.


Jason


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