public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Marek Polacek <polacek@redhat.com>
To: Jakub Jelinek <jakub@redhat.com>
Cc: GCC Patches <gcc-patches@gcc.gnu.org>
Subject: Re: RFC: Monitoring old PRs, new dg directives
Date: Tue, 4 Aug 2020 18:33:10 -0400	[thread overview]
Message-ID: <20200804223310.GK1314761@redhat.com> (raw)
In-Reply-To: <20200730095423.GU2363@tucnak>

On Thu, Jul 30, 2020 at 11:54:23AM +0200, Jakub Jelinek via Gcc-patches wrote:
> On Tue, Jul 28, 2020 at 05:44:47PM -0400, Marek Polacek via Gcc-patches wrote:
> > We will still have a surfeit of bugs that we've given short shrift to, but
> > let's at least automate what we can.  The initial addition of the relevant
> > old(-ish) tests won't of course happen automagically, but it's a price I'm
> > willing to pay.  My goal here isn't merely to reduce the number of open PRs;
> > it is to improve the testing of the compiler overall.
> > 
> > Thoughts?
> 
> Looks useful to me, but I'd think it might be desirable to use separate
> directories for those tests, so that it is more obvious that it is a
> different category of tests.  Now that we use git, just using git mv
> to move them to another place once they are fixed for good (together with
> some dg-* directive tweaks) wouldn't be that much work later.
> 
> So having gcc.dg/unfixed/ , g++.dg/unfixed/ , c-c++-common/unfixed/
> and their torture/ suffixed variants (or better directory name for those)?

Thanks.  I was afraid that it would cause too much friction when you happen
to fix one of the unfixed tests: you will have to find the correct directory
to put the test in and perhaps even rename the test to avoid conflicts with
tests with the same name in the final destination.  But it's also true that
git is much better at moving files, and the extra clarity might be worth the
occasional hassle.  It would also make it easy to skip testing unfixed tests.
dg-ice tests are easy to spot/grep for, but accepts-invalid/rejects-valid are
a different story.

I'll post a v2 patch soon with the unfixed/ dir in mind.

Marek


  reply	other threads:[~2020-08-04 22:33 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-28 21:44 Marek Polacek
2020-07-29  1:37 ` Mike Stump
2020-08-04 21:34   ` Marek Polacek
2020-07-29  3:02 ` Jeff Law
2020-08-04 21:37   ` Marek Polacek
2020-07-29  8:40 ` Richard Sandiford
2020-08-04 21:47   ` Marek Polacek
2020-08-04 22:33     ` Mike Stump
2020-08-05  0:54       ` Marek Polacek
2020-08-05 15:03         ` Nathan Sidwell
2020-08-05 23:29           ` Marek Polacek
2020-08-06 14:01             ` Nathan Sidwell
2020-08-06 22:55               ` Marek Polacek
2020-08-07 13:21                 ` Nathan Sidwell
2020-08-07 14:18                   ` Marek Polacek
2020-08-10  8:48                     ` Richard Sandiford
2020-08-10 12:58                       ` Nathan Sidwell
2020-08-10 21:36                         ` Marek Polacek
2020-08-10 21:30                       ` RFC: Monitoring old PRs, new dg directives [v2] Marek Polacek
2020-08-10 22:58                         ` Mike Stump
2020-11-10 14:15                         ` Thomas Schwinge
2020-11-10 19:07                           ` Marek Polacek
2020-08-07  0:01               ` RFC: Monitoring old PRs, new dg directives Mike Stump
2020-08-07 13:16                 ` Nathan Sidwell
2020-08-10 22:35                   ` Mike Stump
2020-08-05 19:59         ` Marek Polacek
2020-08-05 20:01         ` Mike Stump
2020-08-06 12:27           ` Marek Polacek
2020-08-06 12:30             ` Jakub Jelinek
2020-08-06 12:36               ` Marek Polacek
2020-08-05  8:04     ` Richard Sandiford
2020-08-05 12:59       ` Marek Polacek
2020-07-29 20:37 ` Jason Merrill
2020-08-04 22:08   ` Marek Polacek
2020-08-04 22:45     ` Mike Stump
2020-08-05 12:56       ` Marek Polacek
2020-08-05 19:44         ` Mike Stump
2020-07-29 22:00 ` Martin Sebor
2020-08-04 22:16   ` Marek Polacek
2020-08-04 22:53     ` Mike Stump
2020-08-05  0:59       ` Marek Polacek
2020-07-30  9:08 ` Martin Liška
2020-08-04 22:22   ` Marek Polacek
2020-08-10  3:04     ` Martin Liška
2020-07-30  9:54 ` Jakub Jelinek
2020-08-04 22:33   ` Marek Polacek [this message]
2020-08-05  7:58     ` Richard Sandiford
2020-08-05 13:18       ` Marek Polacek

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=20200804223310.GK1314761@redhat.com \
    --to=polacek@redhat.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=jakub@redhat.com \
    /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).