Re: [RFA] testsuite/gdb.c++/ref-types.exp: use runto

[Fernando Nasser <fnasser@cygnus.com>]

Michael Elizabeth Chastain wrote:
> 
> Mmmm, a philosophical dispute.
> 
> Daniel Berlin writes:
> > They need to be xfail'd for old-abi, but not for new-abi.
> 
> I believe that when gdb has a bug which is under its control, that the
> test suite should issue a FAIL, not an XFAIL.
> 

Yes, but what Dan is trying to say (I guess) is that this is _not_ under GDB's control. I.e., it was not possible for GDB to do the right thing because of insufficient information from the compiler.  Is that right Dan?

If that is the case, it is correct to mark those as XFAILs. Something besides GDB -- something in the execution environment or on another piece of the toolchain -- causes this test to fail and there is not that can be done inside GDB, so the "expected failure".

Maybe you guys can come up with a simple quick test to determine if we are dealing with v2 or v3.  It would be useful to condition tests.


> Here is a gdb log entry for gcc 2.95.2, gdb CVS, Red Hat Linux 7 native,
> stabs:
> 
>   (gdb) print pAe->f()
>   $1 = 134547192
>   (gdb) XFAIL: gdb.c++/virtfunc.exp: print pAe->f()
> 
> If gdb said "I'm sorry, but pAe->f() is too complex for me", I would
> accept that as an XFAIL.  But when gdb prints wrong answers, that should
> be a FAIL.
> 
> I'm interested in other maintainer's opinions on this because I'm
> planning to submit patches to change such XFAIL's to FAIL's, so that
> the test suite can actually report what is broken in C++ support.
> 
> Michael

-- 
Fernando Nasser
Red Hat - Toronto                       E-Mail:  fnasser@redhat.com
2323 Yonge Street, Suite #300
Toronto, Ontario   M4P 2C9