From patchwork Mon Dec  3 10:40:28 2012
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: Fix PR35634
From: Richard Guenther <rguenther@suse.de>
X-Patchwork-Id: 203317
Message-Id: <alpine.LNX.2.00.1212031140170.5402@zhemvz.fhfr.qr>
To: David Edelsohn <dje.gcc@gmail.com>
Cc: GCC Patches <gcc-patches@gcc.gnu.org>,
 "Joseph S. Myers" <joseph@codesourcery.com>
Date: Mon, 3 Dec 2012 11:40:28 +0100 (CET)

On Sat, 1 Dec 2012, David Edelsohn wrote:

> Richard,
> 
> The testcases assume default signed char and fail on systems with
> different semantics.  I believe that both testcases need to declare c
> as signed char to consistently test the desired behavior, right?

Fixed as follows.

Richard.

2012-12-03  Richard Biener  <rguenther@suse.de>

	* gcc.dg/torture/pr35634.c: Use signed char.
	* g++.dg/torture/pr35634.C: Likewise.


Index: gcc/testsuite/gcc.dg/torture/pr35634.c
===================================================================
--- gcc/testsuite/gcc.dg/torture/pr35634.c	(revision 194077)
+++ gcc/testsuite/gcc.dg/torture/pr35634.c	(working copy)
@@ -14,6 +14,6 @@ void foo (int i)
 
 int main ()
 {
-    char c;
+    signed char c;
     for (c = 0; ; c++) foo (c);
 }
Index: gcc/testsuite/g++.dg/torture/pr35634.C
===================================================================
--- gcc/testsuite/g++.dg/torture/pr35634.C	(revision 194077)
+++ gcc/testsuite/g++.dg/torture/pr35634.C	(working copy)
@@ -14,6 +14,6 @@ void foo (int i)
 
 int main ()
 {
-    char c;
+    signed char c;
     for (c = 0; ; c++) foo (c);
 }