]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commitdiff
[gdb/testsuite] Add regression test for PR32158
authorTom de Vries <tdevries@suse.de>
Sat, 14 Sep 2024 12:09:35 +0000 (14:09 +0200)
committerTom de Vries <tdevries@suse.de>
Sat, 14 Sep 2024 12:09:35 +0000 (14:09 +0200)
Consider test-case:
...
namespace ns {
  enum class ec {
    val2 = 2
  };
}

int main () {
  return (int)ns::ec::val2;
}
...
compiled with debug info:
...
$ g++ test.c -g
...

When looking at the cooked index entry for val2 using "maint print objfiles",
we get:
...
    [7] ((cooked_index_entry *) 0x7f8ecc002ef0)
    name:       val2
    canonical:  val2
    qualified:  ns::val2
    DWARF tag:  DW_TAG_enumerator
    flags:      0x0 []
    DIE offset: 0xe9
    parent:     ((cooked_index_entry *) 0x7f8ecc002e90) [ns]
...
which is wrong, there is no source level entity ns::val2.

This is PR symtab/32158.

This is a regression since commit 4e417d7bb1c ("Change handling of
DW_TAG_enumeration_type in DWARF scanner").

Reverting the commit on current trunk fixes the problem, and gets us instead:
...
    [7] ((cooked_index_entry *) 0x7fba70002ef0)
    name:       val2
    canonical:  val2
    qualified:  ns::ec::val2
    DWARF tag:  DW_TAG_enumerator
    flags:      0x0 []
    DIE offset: 0xe9
    parent:     ((cooked_index_entry *) 0x7fba70002ec0) [ec]
...

Add a regression test for this PR in test-case gdb.dwarf2/enum-type-c++.exp.

Tested on x86_64-linux.

Approved-By: Tom Tromey <tom@tromey.com>
Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=32158

gdb/testsuite/gdb.dwarf2/enum-type-c++.cc
gdb/testsuite/gdb.dwarf2/enum-type-c++.exp

index c0ffaad031609c8badc75d1af27e3ca4830ea191..691c7fc7d88e582c12e23e08aefbb70da25d3ffa 100644 (file)
@@ -24,6 +24,12 @@ public:
   };
 };
 
+enum class ec
+{
+  val2 = 2,
+};
 }
 
 int u1 = ns::A::val1;
+
+int u2 = (int)ns::ec::val2;
index 44ad225de02b082c9732418a516bdc3b464ff977..a3dfb4d2f7488d3f762ad7b5a8f05cd2ae9e996f 100644 (file)
@@ -42,3 +42,15 @@ gdb_test_lines "maint print objfiles" \
         ".*"]
 
 gdb_test "print $val1" " = $val1"
+
+# Regression test for PR32158.
+setup_kfail "gdb/32158" *-*-*
+set val2 ns::ec::val2
+gdb_test_lines "maint print objfiles" \
+    "val2 has correct parent" \
+    [multi_line \
+        "" \
+        "$re_ws+qualified:$re_ws+$val2" \
+        ".*"]
+
+gdb_test "print $val2" " = $val2"