From 603f595942ab9c42d6576039033c507191f00610 Mon Sep 17 00:00:00 2001 From: Dodji Seketeli Date: Mon, 23 Mar 2009 23:21:38 +0000 Subject: [PATCH] re PR debug/39524 (Duplicate C++ DW_TAG_variable breaking its DW_AT_location resolution) PR debug/39524 * dwarf2out.c (gen_variable_die): Avoid adding duplicate declaration nodes. * g++.dg/debug/dwarf2/imported-decl-1.C: New test. Co-Authored-By: Jakub Jelinek From-SVN: r145018 --- gcc/ChangeLog | 7 +++++++ gcc/dwarf2out.c | 8 ++++++++ gcc/testsuite/ChangeLog | 6 ++++++ .../g++.dg/debug/dwarf2/imported-decl-1.C | 17 +++++++++++++++++ 4 files changed, 38 insertions(+) create mode 100644 gcc/testsuite/g++.dg/debug/dwarf2/imported-decl-1.C diff --git a/gcc/ChangeLog b/gcc/ChangeLog index c1f7e6f01297..a75ca23f1eaa 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ +2009-03-24 Dodji Seketeli + Jakub Jelinek + + PR debug/39524 + * dwarf2out.c (gen_variable_die): Avoid adding duplicate declaration + nodes. + 2009-03-23 Jakub Jelinek PR c/39495 diff --git a/gcc/dwarf2out.c b/gcc/dwarf2out.c index af5c99484bcf..9bf0803ce570 100644 --- a/gcc/dwarf2out.c +++ b/gcc/dwarf2out.c @@ -13978,6 +13978,14 @@ gen_variable_die (tree decl, tree origin, dw_die_ref context_die) return; } + /* If the compiler emitted a definition for the DECL declaration + and if we already emitted a DIE for it, don't emit a second + DIE for it again. */ + if (old_die + && declaration + && old_die->die_parent == context_die) + return; + var_die = new_die (DW_TAG_variable, context_die, decl); origin_die = NULL; diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index cced718e4218..4f869a0e3e3b 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,9 @@ +2009-03-24 Dodji Seketeli + Jakub Jelinek + + PR debug/39524 + * g++.dg/debug/dwarf2/imported-decl-1.C: New test. + 2009-03-23 Jakub Jelinek PR c/39495 diff --git a/gcc/testsuite/g++.dg/debug/dwarf2/imported-decl-1.C b/gcc/testsuite/g++.dg/debug/dwarf2/imported-decl-1.C new file mode 100644 index 000000000000..bfdb4f8f5a04 --- /dev/null +++ b/gcc/testsuite/g++.dg/debug/dwarf2/imported-decl-1.C @@ -0,0 +1,17 @@ +// PR debug/39524 +// { dg-do compile } +// { dg-options "-gdwarf-2 -dA -O0 -fno-merge-debug-strings" } + +namespace A +{ + static int var2 = 2; +} + +int +func () +{ + using A::var2; + return var2; +} + +// { dg-final { scan-assembler-times "var2\[^\n\r\]*DW_AT_name" 1 } } -- 2.47.3