[PATCH2 PR gdb/18071] aarch64: "info addr" command can't resolve TLS variables
Simon Marchi
simon.marchi@polymtl.ca
Tue Mar 27 03:31:00 GMT 2018
On 2018-03-26 17:54, Weimin Pan wrote:
> Running the test case with upstream gdb shows two failures:
>
> (1) Receiving different error messages when printing TLS variable
> before
> program runs - because the ARM compiler does not emit dwarf
> attribute
> DW_AT_location for TLS, the result is expected and the baseline may
> need to be changed for aarch64.
>
> (2) Using "info address" command on C++ static TLS object resulted in
> "symbol unresolved" error - below is a snippet from the test case:
>
> class K {
> public:
> static __thread int another_thread_local;
> };
>
> __thread int K::another_thread_local;
>
> (gdb) info address K::another_thread_local
> Symbol "K::another_thread_local" is unresolved.
>
> This patch contains fix for (2).
>
> Function info_address_command() handles the "info address" command and
> calls lookup_minimal_symbol_and_objfile() to find sym's symbol entry in
> minimal symbol table if SYMBOL_COMPUTED_OPS (sym) is false. Problem is
> that function lookup_minimal_symbol_and_objfile() only looked up an
> objfile's minsym ordinary hash table, not its demangled hash table,
> which
> was the reason why the C++ name was not found.
>
> The fix is to replace the lookup_minimal_symbol_and_objfile() call with
> the lookup_bound_minimal_symbol() call, which looks up entries in both
> minsym's hash tables, via lookup_minimal_symbol(), to find symbol entry
> that's associated with the demangled name.
Hi Weimin,
I don't know if it was clear, but I pushed this patch already:
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=commit;h=bce02d8884d6baa72c537d0d7c59f924cb290799
Removing lookup_minimal_symbol_and_objfile and replacing it with
lookup_bound_minimal_symbol should be done as a separate patch (with a
relevant title that describes the change). Can you do that? There are
just a few callers of lookup_minimal_symbol_and_objfile that will need
to be updated.
Simon
More information about the Gdb-patches
mailing list