libiberty: Invoke D demangler when --format=auto

Investigating GDB PR d/31580 showed that the libiberty demangler
doesn't automatically demangle D mangled names.  However, I think it
should -- like C++ and Rust (new-style), D mangled names are readily
distinguished by the leading "_D", and so the likelihood of confusion
is low.  The other non-"auto" cases in this code are Ada (where the
encoded form could more easily be confused by ordinary programs) and
Java (which is long gone, but which also shared the C++ mangling and
thus was just an output style preference).

This patch also fixed another GDB bug, though of course that part
won't apply to the GCC repository.

Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=31580
Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=30276

libiberty
	* cplus-dem.c (cplus_demangle): Try the D demangler with
	"auto" format.
	* testsuite/d-demangle-expected: Add --format=auto test.
This commit is contained in:
Tom Tromey
2024-03-30 13:48:30 -06:00
parent 251cedaeb5
commit b1741ab0da
3 changed files with 7 additions and 4 deletions

View File

@@ -79,10 +79,8 @@ if {[gdb_start_cmd] < 0} {
return -1
}
# We should probably have "D main" instead of "_Dmain" here, filed PR30276
# '[gdb/symtab] function name is _Dmain instead of "D main"' about that.
gdb_test "" \
"in _Dmain \\(\\)" \
"in D main \\(\\)" \
"start"
gdb_test "show language" {"auto; currently d".}

View File

@@ -186,7 +186,7 @@ cplus_demangle (const char *mangled, int options)
if (GNAT_DEMANGLING)
return ada_demangle (mangled, options);
if (DLANG_DEMANGLING)
if (DLANG_DEMANGLING || AUTO_DEMANGLING)
{
ret = dlang_demangle (mangled, options);
if (ret)

View File

@@ -1470,3 +1470,8 @@ demangle.anonymous
--format=dlang
_D8demangle9anonymous03fooZ
demangle.anonymous.foo
#
# Test that 'auto' works.
--format=auto
_D8demangle9anonymous03fooZ
demangle.anonymous.foo