From 53097e8c986887a8f93cabbc6b96dac85e6698c8 Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Tue, 27 Jul 2010 16:11:03 +0000 Subject: [PATCH] * dwarf2loc.c (locexpr_describe_location_piece): Also recognize TLS with DW_OP_const4u or DW_OP_const8u. --- gdb/ChangeLog | 5 +++++ gdb/dwarf2loc.c | 14 +++++++++----- 2 files changed, 14 insertions(+), 5 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 1585b526993..f22ce9457b2 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2010-07-27 Tom Tromey + + * dwarf2loc.c (locexpr_describe_location_piece): Also recognize + TLS with DW_OP_const4u or DW_OP_const8u. + 2010-07-22 Pedro Alves * NEWS: Mention target reported shared libraries support by diff --git a/gdb/dwarf2loc.c b/gdb/dwarf2loc.c index 2a8e557f284..f59bc4052fa 100644 --- a/gdb/dwarf2loc.c +++ b/gdb/dwarf2loc.c @@ -2016,15 +2016,19 @@ locexpr_describe_location_piece (struct symbol *symbol, struct ui_file *stream, DW_AT_location : 10 byte block: 3 4 0 0 0 0 0 0 0 e0 (DW_OP_addr: 4; DW_OP_GNU_push_tls_address) - + 0x3 is the encoding for DW_OP_addr, which has an operand as long as the size of an address on the target machine (here is 8 - bytes). 0xe0 is the encoding for DW_OP_GNU_push_tls_address. - The operand represents the offset at which the variable is within - the thread local storage. */ + bytes). Note that more recent version of GCC emit DW_OP_const4u + or DW_OP_const8u, depending on address size, rather than + DW_OP_addr. 0xe0 is the encoding for + DW_OP_GNU_push_tls_address. The operand represents the offset at + which the variable is within the thread local storage. */ else if (data + 1 + addr_size < end - && data[0] == DW_OP_addr + && (data[0] == DW_OP_addr + || (addr_size == 4 && data[0] == DW_OP_const4u) + || (addr_size == 8 && data[0] == DW_OP_const8u)) && data[1 + addr_size] == DW_OP_GNU_push_tls_address && piece_end_p (data + 2 + addr_size, end)) { -- 2.47.2