public inbox for gdb-cvs@sourceware.org
help / color / mirror / Atom feed
* [binutils-gdb] gdb.dlang/demangle.exp: update expected output for _D8demangle4testFnZv
@ 2022-01-14 20:09 Simon Marchi
  0 siblings, 0 replies; only message in thread
From: Simon Marchi @ 2022-01-14 20:09 UTC (permalink / raw)
  To: gdb-cvs

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=513569fbb7c9ee4f7767d7485ea0e44224aa6fdf

commit 513569fbb7c9ee4f7767d7485ea0e44224aa6fdf
Author: Simon Marchi <simon.marchi@efficios.com>
Date:   Fri Jan 14 15:09:32 2022 -0500

    gdb.dlang/demangle.exp: update expected output for _D8demangle4testFnZv
    
    Since commit ce2d3708bc8b ("Synchronize binutils libiberty sources with
    gcc version."), I see this failure:
    
        demangle _D8demangle4testFnZv^M
        demangle.test(typeof(null))^M
        (gdb) FAIL: gdb.dlang/demangle.exp: _D8demangle4testFnZv
    
    The commit imported the commit 0e32a5aa8bc9 ("libiberty: Add support for
    D `typeof(*null)' types") from the gcc repository.  That commit includes
    an update to libiberty/testsuite/d-demangle-expected, which updates a
    test for the exact same mangled name:
    
         _D8demangle4testFnZv
        -demangle.test(none)
        +demangle.test(typeof(null))
    
    I don't know anything about D, but give that the change was made by Iain
    Buclaw, the D language maintainer, I trust him on that.
    
    Fix our test by updating the expected output in the same way.
    
    Note: it's not really useful to have all these D demangling tests in the
    GDB testsuite, since there are demangling tests in libiberty.  We should
    consider removing them, but we first need to make sure that everything
    that is covered in gdb/testsuite/gdb.dlang/demangle.exp is also covered
    in libiberty/testsuite/d-demangle-expected.
    
    Change-Id: If2b290ea8367b8e1e0b90b20d4a6e0bee517952d

Diff:
---
 gdb/testsuite/gdb.dlang/demangle.exp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gdb/testsuite/gdb.dlang/demangle.exp b/gdb/testsuite/gdb.dlang/demangle.exp
index ebc2487fd1b..d45437c72cf 100644
--- a/gdb/testsuite/gdb.dlang/demangle.exp
+++ b/gdb/testsuite/gdb.dlang/demangle.exp
@@ -44,7 +44,7 @@ proc test_d_demangling {} {
     test_demangling "_D8demangle4testFkZv" "demangle.test(uint)"
     test_demangling "_D8demangle4testFlZv" "demangle.test(long)"
     test_demangling "_D8demangle4testFmZv" "demangle.test(ulong)"
-    test_demangling "_D8demangle4testFnZv" "demangle.test(none)"
+    test_demangling "_D8demangle4testFnZv" "demangle.test(typeof(null))"
     test_demangling "_D8demangle4testFoZv" "demangle.test(ifloat)"
     test_demangling "_D8demangle4testFpZv" "demangle.test(idouble)"
     test_demangling "_D8demangle4testFqZv" "demangle.test(cfloat)"


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2022-01-14 20:09 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-14 20:09 [binutils-gdb] gdb.dlang/demangle.exp: update expected output for _D8demangle4testFnZv Simon Marchi

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).