Re: [PATCH 1/2] testsuite fixes for 16-bit targets

[Rask Ingemann Lambertsen <rask at sygehus dot dk>]

On Wed, Jul 04, 2007 at 09:00:29PM +0100, Richard Sandiford wrote:
> Rask Ingemann Lambertsen <rask@sygehus.dk> writes:
> >    The attached patch fixes 36 testsuite failures on m32c-unknown-elf. The
> > fix is to change "int" to "long int" in several places that need more than
> > 16 bits.
> 
> I don't think we should do that unconditionally.  It changes what's
> being tested on LP64 targets in a major way.  It seems better just
> to skip the tests for 16-bit targets, and perhaps add "long int" copies
> if you think that what they're testing is useful for 16-bit targets.

   Here is a patch to skip the tests.

2007-07-13  Rask Ingemann Lambertsen  <rask@sygehus.dk>

	* gcc.dg/tree-ssa/ltrans-5.c: Only run test on targets which
	support large arrays.
	* gcc.dg/pr27959.c: Only run test if an int is at least 32 bits wide.
	* gcc.dg/20061109-1.c: Likewise.
	* gcc.c-torture/execute/20061101-1.x: Likewise.
	* gcc.c-torture/execute/pr29797-1.x: Likewise.
	* gcc.c-torture/execute/pr31448.x: Likewise.

Index: gcc.dg/tree-ssa/ltrans-5.c
===================================================================
--- gcc.dg/tree-ssa/ltrans-5.c	(revision 125984)
+++ gcc.dg/tree-ssa/ltrans-5.c	(working copy)
@@ -1,4 +1,4 @@
-/* { dg-do compile } */ 
+/* { dg-do compile { target { size32plus } } } */ 
 /* { dg-options "-O2 -ftree-loop-linear -fdump-tree-ltrans-all" } */
 
 int foo ()
Index: gcc.dg/pr27959.c
===================================================================
--- gcc.dg/pr27959.c	(revision 125984)
+++ gcc.dg/pr27959.c	(working copy)
@@ -1,5 +1,5 @@
 /* PR middle-end/27959 */
-/* { dg-do run } */
+/* { dg-do run { target { int32plus } } } */
 /* { dg-options "-O2" } */
 /* { dg-options "-O2 -mtune=z990" { target s390*-*-* } } */
 
Index: gcc.dg/20061109-1.c
===================================================================
--- gcc.dg/20061109-1.c	(revision 125984)
+++ gcc.dg/20061109-1.c	(working copy)
@@ -1,4 +1,4 @@
-/* { dg-do compile } */
+/* { dg-do compile { target { int32plus } } } */
 /* { dg-options "-O1" } */
 
 /* This assignment might be implemented with a mem copy from the
Index: gcc.c-torture/execute/20061101-1.x
===================================================================
--- gcc.c-torture/execute/20061101-1.x	(revision 0)
+++ gcc.c-torture/execute/20061101-1.x	(revision 0)
@@ -0,0 +1,7 @@
+load_lib target-supports.exp
+
+if { [check_effective_target_int32plus] } {
+	return 0
+}
+
+return 1;
Index: gcc.c-torture/execute/pr29797-1.x
===================================================================
--- gcc.c-torture/execute/pr29797-1.x	(revision 0)
+++ gcc.c-torture/execute/pr29797-1.x	(revision 0)
@@ -0,0 +1,7 @@
+load_lib target-supports.exp
+
+if { [check_effective_target_int32plus] } {
+	return 0
+}
+
+return 1;
Index: gcc.c-torture/execute/pr31448.x
===================================================================
--- gcc.c-torture/execute/pr31448.x	(revision 0)
+++ gcc.c-torture/execute/pr31448.x	(revision 0)
@@ -0,0 +1,7 @@
+load_lib target-supports.exp
+
+if { [check_effective_target_int32plus] } {
+	return 0
+}
+
+return 1;

-- 
Rask Ingemann Lambertsen