This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] gcc.c-torture/ cleanup

From: Jeff Law <law at redhat dot com>
To: Marek Polacek <polacek at redhat dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
Cc: Mike Stump <mikestump at comcast dot net>, "Joseph S. Myers" <joseph at codesourcery dot com>
Date: Tue, 30 Sep 2014 10:41:48 -0600
Subject: Re: [PATCH] gcc.c-torture/ cleanup
Authentication-results: sourceware.org; auth=none
References: <20140930152205 dot GD32230 at redhat dot com>

On 09/30/14 09:22, Marek Polacek wrote:

I did this as a part of preparing the testsuite to cope with the
(possible) gnu11 default.  But I think it's a reasonable cleanup
on its own.  With gnu11, we'd start to warn about defaulting to
int, missing function declarations, and functions without return
type.  I added -fgnu89-inline when a test relies on a gnu89 inline
semantics, and -std=gnu89 if a test relies on gnu89 standard.

I have patches that cover the rest of C testsuite, but let's do this
piecewise.

Tested on x86_64-linux: vanilla results == results with this patch ==
results with this patch and gnu11 as a default.

Does this approach make sense?

2014-09-30  Marek Polacek  <polacek@redhat.com>

	* gcc.c-torture/compile/20000120-2.c: Use -fgnu89-inline.
	* gcc.c-torture/compile/20011119-1.c: Likewise.
	* gcc.c-torture/compile/20011119-2.c: Likewise.
	* gcc.c-torture/compile/20021120-1.c: Likewise.
	* gcc.c-torture/compile/20021120-2.c: Likewise.
	* gcc.c-torture/compile/20050215-1.c: Likewise.
	* gcc.c-torture/compile/20050215-2.c: Likewise.
	* gcc.c-torture/compile/20050215-3.c: Likewise.
	* gcc.c-torture/compile/pr37669.c: Likewise.
	* gcc.c-torture/execute/20020107-1.c: Likewise.
	* gcc.c-torture/execute/restrict-1.c: Likewise.
	* gcc.c-torture/compile/20090721-1.c: Fix defaulting to int.
	* gcc.c-torture/execute/930529-1.c: Likewise.
	* gcc.c-torture/execute/920612-1.c: Likewise.
	* gcc.c-torture/execute/920711-1.c: Likewise.
	* gcc.c-torture/execute/990127-2.c: Likewise.
	* gcc.c-torture/execute/pr40386.c: Likewise.
	* gcc.c-torture/execute/pr57124.c: Likewise.
	* gcc.c-torture/compile/pr34808.c: Add function declarations.
	* gcc.c-torture/compile/pr42299.c: Likewise.
	* gcc.c-torture/compile/pr48517.c: Use -std=gnu89.
	* gcc.c-torture/compile/simd-6.c: Likewise.
	* gcc.c-torture/execute/pr53645-2.c: Likewise.
	* gcc.c-torture/execute/pr53645.c: Likewise.
	* gcc.c-torture/execute/20001121-1.c: Use -fgnu89-inline.  Add function
	declarations.
	* gcc.c-torture/execute/980608-1.c: Likewise.
	* gcc.c-torture/execute/bcp-1.c: Likewise.
	* gcc.c-torture/execute/p18298.c: Likewise.
	* gcc.c-torture/execute/unroll-1.c: Likewise.
	* gcc.c-torture/execute/va-arg-7.c: Likewise.
	* gcc.c-torture/execute/va-arg-8.c: Likewise.
	* gcc.c-torture/execute/930526-1.c: Use -fgnu89-inline.  Add function
	declarations.  Fix defaulting to int.
	* gcc.c-torture/execute/961223-1.c: Likewise.
	* gcc.c-torture/execute/loop-2c.c: Use -fgnu89-inline and
	-Wno-pointer-to-int-cast.  Fix defaulting to int.

OK.
Jeff

References:
- [PATCH] gcc.c-torture/ cleanup
  - From: Marek Polacek

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]