From: Yao Qi <qiyaoltc@gmail.com>
To: Andrew Burgess <andrew.burgess@embecosm.com>
Cc: "gdb-patches@sourceware.org" <gdb-patches@sourceware.org>
Subject: Re: [PATCH] gdb: Use UNSUPPORTED not FAIL for unsupported target features
Date: Wed, 08 Jun 2016 14:59:00 -0000 [thread overview]
Message-ID: <CAH=s-POHiPAtPmB_YHvmgT-m8rQyooOz4qd=0ndGWHjPc5RC8A@mail.gmail.com> (raw)
In-Reply-To: <20160608114017.GB26734@embecosm.com>
On Wed, Jun 8, 2016 at 12:40 PM, Andrew Burgess
> diff --git a/gdb/testsuite/gdb.base/callfuncs.exp b/gdb/testsuite/gdb.base/callfuncs.exp
> index fbe896c..1ec33d8 100644
> --- a/gdb/testsuite/gdb.base/callfuncs.exp
> +++ b/gdb/testsuite/gdb.base/callfuncs.exp
> @@ -26,8 +26,7 @@ if [support_complex_tests] {
> # Some targets can't do function calls, so don't even bother with this
> # test.
> if [target_info exists gdb,cannot_call_functions] {
> - setup_xfail "*-*-*" 2416
> - fail "This target can not call functions"
> + unsupported "This target can not call functions"
> continue
> }
>
That change makes sense to me. UNSUPPORTED is better XFAIL.
There are many instances,
./gdb.base/unwindonsignal.exp: setup_xfail "*-*-*" 2416
./gdb.base/call-signal-resume.exp: setup_xfail "*-*-*" 2416
./gdb.base/call-rt-st.exp: setup_xfail "*-*-*" 2416
./gdb.base/call-ar-st.exp: setup_xfail "*-*-*" 2416
./gdb.base/ptype.exp: setup_xfail "*-*-*" 2416
./gdb.base/callexit.exp: setup_xfail "*-*-*" 2416
./gdb.base/callfuncs.exp: setup_xfail "*-*-*" 2416
./gdb.base/nodebug.exp: setup_xfail "*-*-*" 2416
./gdb.base/call-strs.exp: setup_xfail "*-*-*" 2416
./gdb.base/printcmds.exp: setup_xfail "*-*-*" 2416
./gdb.base/printcmds.exp: setup_xfail "*-*-*" 2416
./gdb.threads/thread-unwindonsignal.exp: setup_xfail "*-*-*" 2416
./gdb.threads/hand-call-in-threads.exp: setup_xfail "*-*-*" 2416
./gdb.threads/interrupted-hand-call.exp: setup_xfail "*-*-*" 2416
./gdb.cp/templates.exp: setup_xfail "*-*-*" 2416
./gdb.cp/gdb2495.exp: setup_xfail "*-*-*" 2416
./gdb.cp/virtfunc.exp: setup_xfail "*-*-*" 2416
so I think we need to clean them up too. I don't think 2416 is the PR number
in sourceware.org bugzilla, so we don't loose any information after
the cleanup. The patch can go in without another review.
--
Yao (齐尧)
next prev parent reply other threads:[~2016-06-08 14:59 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-08 9:26 Andrew Burgess
2016-06-08 10:25 ` Yao Qi
2016-06-08 11:40 ` Andrew Burgess
2016-06-08 14:59 ` Yao Qi [this message]
2016-06-13 14:29 ` [PUSHED] " Andrew Burgess
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAH=s-POHiPAtPmB_YHvmgT-m8rQyooOz4qd=0ndGWHjPc5RC8A@mail.gmail.com' \
--to=qiyaoltc@gmail.com \
--cc=andrew.burgess@embecosm.com \
--cc=gdb-patches@sourceware.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).