This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Question about a varasm change
- To: Jason Merrill <jason at redhat dot com>
- Subject: Re: Question about a varasm change
- From: Bernd Schmidt <bernds at redhat dot co dot uk>
- Date: Mon, 18 Sep 2000 10:32:20 +0100 (BST)
- cc: gcc at gcc dot gnu dot org, gcc-patches at gcc dot gnu dot org
On 18 Sep 2000, Jason Merrill wrote:
> The change fixes DECL_ASSEMBLER_NAME so that it actually corresponds to the
> name used in assembler output, as documented. If the stabs backend expects
> it to be the unmangled name from the source, it's looking in the wrong
> place; that's what DECL_NAME is for.
Thanks. Is this patch OK? It fixes a bunch of gdb failures.
Bernd
* dbxout.c (dbxout_symbol_name): Use DECL_NAME rather than
DECL_ASSEMBLER_NAME.
Index: dbxout.c
===================================================================
RCS file: /cvs/gcc/egcs/gcc/dbxout.c,v
retrieving revision 1.60
diff -u -p -r1.60 dbxout.c
--- dbxout.c 2000/07/23 17:53:21 1.60
+++ dbxout.c 2000/09/18 09:31:12
@@ -2219,7 +2219,7 @@ dbxout_symbol_name (decl, suffix, letter
class member, we must put out the mangled name instead of the
DECL_NAME. Note also that static member (variable) names DO NOT begin
with underscores in .stabs directives. */
- const char *name = IDENTIFIER_POINTER (DECL_ASSEMBLER_NAME (decl));
+ const char *name = IDENTIFIER_POINTER (DECL_NAME (decl));
if (name == 0)
name = "(anon)";
fprintf (asmfile, "%s \"%s%s:", ASM_STABS_OP, name,