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: [PATCH]Add symbol whose field 'has_type' has been set to partial symbol table


>>>>> ">" == hex  <heixia108@gmail.com> writes:

>> // Then compile it with `gcc -g -c test.c`(For the lastest GCC, we
>> need use 'gcc -g -O1 -c test.' to get the following DIE)
>> We could see the DIE of 'var' is as following:
>>  <1><25>: Abbrev Number: 2 (DW_TAG_variable)
>>     <26>   DW_AT_name        : var
>>     <2a>   DW_AT_decl_file   : 1
>>     <2b>   DW_AT_decl_line   : 1
>>     <2c>   DW_AT_type        : <0x31>
>>     <30>   DW_AT_const_value : 3

>> Latest GDB will not add it to partial symbol table because its symbol
>> satisfies 'pdi->d.locdesc == NULL'.  I think we need add it to partial
>> symbol table.

This patch seems reasonable to me, but I think it needs a test case.
This should be easy to write using the DWARF assembler in the test suite.

Tom


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