compare1.c: Skip on arm*-*-eabi* arm*-*-symbianelf*.

	* gcc.dg/compare1.c: Skip on arm*-*-eabi* arm*-*-symbianelf*.
	* gcc.dg/compare9.c: New.

From-SVN: r101604

3 files changed, 50 insertions, 0 deletions

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 54fc160..1f18a29 100644
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2005-07-04  Kazu Hirata  <kazu@codesourcery.com>
+
+	* gcc.dg/compare1.c: Skip on arm*-*-eabi* arm*-*-symbianelf*.
+	* gcc.dg/compare9.c: New.
+
 2005-07-04  Francois-Xavier Coudert  <coudert@clipper.ens.fr>
 
 	* gfortran.dg/io_invalid_1.f90: Fix expected error message.
diff --git a/gcc/testsuite/gcc.dg/compare1.c b/gcc/testsuite/gcc.dg/compare1.c
index 44c4df8..0604927 100644
--- a/gcc/testsuite/gcc.dg/compare1.c
+++ b/gcc/testsuite/gcc.dg/compare1.c
@@ -3,6 +3,9 @@
 
 /* { dg-do compile } */
 /* { dg-options "-Wsign-compare" } */
+/* This test would fail on targets with short enums being default.  See
+   compare9.c.  */
+/* { dg-skip-if "" { arm*-*-eabi* arm*-*-symbianelf* }  { "*" } { "" } } */
 
 int tf = 1;
 
diff --git a/gcc/testsuite/gcc.dg/compare9.c b/gcc/testsuite/gcc.dg/compare9.c
new file mode 100644
index 0000000..e46e064
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/compare9.c
@@ -0,0 +1,42 @@
+/* Test for a bogus warning on comparison between signed and unsigned.
+   This was inspired by code in gcc.  This testcase is identical to
+   compare1.c except that we add -fno-short-enums to accomodate
+   targets with short enums being default, such as arm*-*-eabi* and
+   arm*-*-symbianelf*.  */
+
+/* { dg-do compile } */
+/* { dg-options "-fno-short-enums -Wsign-compare" } */
+
+int tf = 1;
+
+/* This enumeration has an explicit negative value and is therefore signed.  */
+enum mm1 
+{
+  VOID, SI, DI, MAX = -1
+};
+
+/* This enumeration fits entirely in a signed int, but is unsigned anyway.  */
+enum mm2
+{
+  VOID2, SI2, DI2, MAX2
+};
+
+int f(enum mm1 x)
+{
+  return x == (tf?DI:SI); /* { dg-bogus "signed and unsigned" "case 1" } */
+}
+
+int g(enum mm1 x)
+{
+  return x == (tf?DI:-1); /* { dg-bogus "signed and unsigned" "case 2" } */
+}
+
+int h(enum mm2 x)
+{
+  return x == (tf?DI2:SI2); /* { dg-bogus "signed and unsigned" "case 3" } */
+}
+
+int i(enum mm2 x)
+{
+  return x == (tf?DI2:-1); /* { dg-warning "signed and unsigned" "case 4" } */
+}