public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* Duplicate testcases
@ 2004-06-11 17:42 Joseph S. Myers
  2004-06-11 20:11 ` Mike Stump
  2004-06-27 21:25 ` Joseph S. Myers
  0 siblings, 2 replies; 3+ messages in thread
From: Joseph S. Myers @ 2004-06-11 17:42 UTC (permalink / raw)
  To: gcc-patches

There are several testcases that are exact duplicates (as found by
identical MD5 checksums) and don't appear to be handled differently (e.g.
it's OK to have the same text for a testcase used for both C and C++).  I
propose removing one of each pair of those under gcc/testsuite (ChangeLog
entry below).  I believe this is obviously correct, but in case some of
these testcases are in fact handled differently or there's a reason in
some case to remove the other one of the pair of duplicates, I won't apply
this immediately.

2004-06-11  Joseph S. Myers  <jsm@polyomino.org.uk>

	* gcc.c-torture/compile/20021123-1.c: Remove duplicate of
	20021110.c.
	* gcc.c-torture/compile/20021123-4.c: Remove duplicate of
	20021119-1.c.
	* gcc.dg/vmx/5-05.c, gcc.dg/vmx/5-06.c, gcc.dg/vmx/5-07.c,
	gcc.dg/vmx/5-08.c: Remove duplicates of 5-01.c, 5-02.c, 5-03.c,
	5-04.c.
	* gcc.dg/builtins-42.c: Remove duplicate of builtins-41.c.
	* g++.dg/other/new1.C: Remove duplicate of g++.dg/init/new1.C.
	* g++.dg/init/elide3.C: Remove duplicate of elide2.C.
	* g++.old-deja/g++.law/enum8.C: Remove duplicate of
	g++.brendan/enum14.C.
	* g++.old-deja/g++.mike/ns8.C: Remove duplicate of ns6.C
	* g++.old-deja/g++.ns/alias5.C: Remove duplicate of alias1.C.
	* g++.old-deja/g++.other/virtual1.C: Remove duplicate of
	g++.mike/virt1.C.
	* g++.old-deja/g++.pt/memtemp27.C: Remove duplicate of
	memtemp24.C.

-- 
Joseph S. Myers
jsm@polyomino.org.uk

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Duplicate testcases
  2004-06-11 17:42 Duplicate testcases Joseph S. Myers
@ 2004-06-11 20:11 ` Mike Stump
  2004-06-27 21:25 ` Joseph S. Myers
  1 sibling, 0 replies; 3+ messages in thread
From: Mike Stump @ 2004-06-11 20:11 UTC (permalink / raw)
  To: Joseph S. Myers; +Cc: gcc-patches

On Friday, June 11, 2004, at 09:12  AM, Joseph S. Myers wrote:
> 	* g++.dg/other/new1.C: Remove duplicate of g++.dg/init/new1.C.
> 	* g++.dg/init/elide3.C: Remove duplicate of elide2.C.
> 	* g++.old-deja/g++.law/enum8.C: Remove duplicate of
> 	g++.brendan/enum14.C.

These all seem safe enough.

> 	* g++.old-deja/g++.mike/ns8.C: Remove duplicate of ns6.C

I was curious about this so I researched this, no rcs files for the 
testsuite back in oldgcc, and I can't get at devo to check.  Oh well... 
  I'd bet that someone edited the testcase after 1.1 in devo, and the 
cases just happened to match after...

> 	* g++.old-deja/g++.ns/alias5.C: Remove duplicate of alias1.C.
> 	* g++.old-deja/g++.other/virtual1.C: Remove duplicate of
> 	g++.mike/virt1.C.
> 	* g++.old-deja/g++.pt/memtemp27.C: Remove duplicate of
> 	memtemp24.C.

These all seem safe enough.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Duplicate testcases
  2004-06-11 17:42 Duplicate testcases Joseph S. Myers
  2004-06-11 20:11 ` Mike Stump
@ 2004-06-27 21:25 ` Joseph S. Myers
  1 sibling, 0 replies; 3+ messages in thread
From: Joseph S. Myers @ 2004-06-27 21:25 UTC (permalink / raw)
  To: gcc-patches

I've now applied this patch, given there were no comments to indicate that
removal of any of these duplicates wasn't in fact obviously correct.
Hopefully this will cut a second or two from testsuite run times.

> 2004-06-11  Joseph S. Myers  <jsm@polyomino.org.uk>
> 
> 	* gcc.c-torture/compile/20021123-1.c: Remove duplicate of
> 	20021110.c.
> 	* gcc.c-torture/compile/20021123-4.c: Remove duplicate of
> 	20021119-1.c.
> 	* gcc.dg/vmx/5-05.c, gcc.dg/vmx/5-06.c, gcc.dg/vmx/5-07.c,
> 	gcc.dg/vmx/5-08.c: Remove duplicates of 5-01.c, 5-02.c, 5-03.c,
> 	5-04.c.
> 	* gcc.dg/builtins-42.c: Remove duplicate of builtins-41.c.
> 	* g++.dg/other/new1.C: Remove duplicate of g++.dg/init/new1.C.
> 	* g++.dg/init/elide3.C: Remove duplicate of elide2.C.
> 	* g++.old-deja/g++.law/enum8.C: Remove duplicate of
> 	g++.brendan/enum14.C.
> 	* g++.old-deja/g++.mike/ns8.C: Remove duplicate of ns6.C
> 	* g++.old-deja/g++.ns/alias5.C: Remove duplicate of alias1.C.
> 	* g++.old-deja/g++.other/virtual1.C: Remove duplicate of
> 	g++.mike/virt1.C.
> 	* g++.old-deja/g++.pt/memtemp27.C: Remove duplicate of
> 	memtemp24.C.

-- 
Joseph S. Myers               http://www.srcf.ucam.org/~jsm28/gcc/
    jsm@polyomino.org.uk (personal mail)
    jsm28@gcc.gnu.org (Bugzilla assignments and CCs)

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2004-06-27 18:58 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-06-11 17:42 Duplicate testcases Joseph S. Myers
2004-06-11 20:11 ` Mike Stump
2004-06-27 21:25 ` Joseph S. Myers

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).