diff mbox

[testsuite] : Skip some tests for avr

Message ID 4DADC684.5030908@gjlay.de
State New
Headers show

Commit Message

Georg-Johann Lay April 19, 2011, 5:29 p.m. UTC 
Hans-Peter Nilsson schrieb:
> On Tue, 19 Apr 2011, Georg-Johann Lay wrote:
>> This patchlet skips some tests for avr because int is just 16 bits there.
> 
>> 2011-04-19  Georg-Johann Lay  <avr@gjlay.de>
>>
>> 	* gcc.c-torture/compile/pr43191.c: Skip avr due to 16-bit int.
>> 	* gcc.dg/torture/pr43165.c: Ditto.
>> 	* gcc.dg/torture/pr47228.c: Ditto.
>> 	* gcc.dg/tree-ssa/pr45144.c: Ditto.
>> 	* gcc.dg/ipa/pr45644.c: Ditto.
>>
> 
> (Cutnpasted instead of quoting because alpine can't quote
> attached text; please inline patches.)
> 
> Index: gcc.c-torture/compile/pr43191.c
> ===================================================================
> --- gcc.c-torture/compile/pr43191.c     (Revision 172597)
> +++ gcc.c-torture/compile/pr43191.c     (Arbeitskopie)
> @@ -1,4 +1,4 @@
> -/* { dg-skip-if "Ints are 16 bits" { "pdp11-*-*" } { "*" } { "" } } */
> +/* { dg-skip-if "Ints are 16 bits" { "avr-*-*" "pdp11-*-*" } { "*" } { "" } }*/
> 
> That pdp11 stanza was a bad precedent.  Please instead use one of
> 
> /* { dg-skip-if "Ints are 16 bits" { ! int32plus } { "*" } { "" } } */
> or
> /* { dg-require-effective-target int32plus } */
> or
> /* { dg-do X { target int32plus } } */
> (where X = {compile, run, assemble})
> 
> so we don't have to keep a steadily increasing target list
> (decreasing only when targets are removed).
> 
> brgds, H-P

Thanks for that hint. There are some more test cases that imply int >=
32 bit

Johann

2011-04-19  Georg-Johann Lay  <avr@gjlay.de>

	* gcc.dg/pr42629.c: Add dg-require-effective-target int32plus
	* gcc.c-torture/execute/cmpsi-2.c: Ditto
	* gcc.c-torture/execute/pr45262.c: Ditto
	* gcc.dg/torture/pr43165.c: Ditto.
	* gcc.dg/torture/pr47228.c: Ditto.
	* gcc.dg/tree-ssa/pr45144.c: Ditto.
	* gcc.dg/ipa/pr45644.c: Ditto.
	* gcc.c-torture/compile/pr43191.c: Ditto. Remove dg-skip-if for
	PDP11.

Comments

Mike Stump April 20, 2011, 5:12 a.m. UTC | #1 
On Apr 19, 2011, at 10:29 AM, Georg-Johann Lay wrote:
> Thanks for that hint. There are some more test cases that imply int >=
> 32 bit
> 
> Johann
> 
> 2011-04-19  Georg-Johann Lay  <avr@gjlay.de>
> 
> 	* gcc.dg/pr42629.c: Add dg-require-effective-target int32plus
> 	* gcc.c-torture/execute/cmpsi-2.c: Ditto
> 	* gcc.c-torture/execute/pr45262.c: Ditto
> 	* gcc.dg/torture/pr43165.c: Ditto.
> 	* gcc.dg/torture/pr47228.c: Ditto.
> 	* gcc.dg/tree-ssa/pr45144.c: Ditto.
> 	* gcc.dg/ipa/pr45644.c: Ditto.
> 	* gcc.c-torture/compile/pr43191.c: Ditto. Remove dg-skip-if for
> 	PDP11.


Ok, thanks.
diff mbox

Patch

Index: gcc.c-torture/execute/cmpsi-2.c
===================================================================
--- gcc.c-torture/execute/cmpsi-2.c	(Revision 172597)
+++ gcc.c-torture/execute/cmpsi-2.c	(Arbeitskopie)
@@ -1,3 +1,5 @@ 
+/* { dg-require-effective-target int32plus } */
+
 #define F 140
 #define T 13
 
Index: gcc.c-torture/execute/pr45262.c
===================================================================
--- gcc.c-torture/execute/pr45262.c	(Revision 172597)
+++ gcc.c-torture/execute/pr45262.c	(Arbeitskopie)
@@ -1,4 +1,5 @@ 
 /* PR middle-end/45262 */
+/* { dg-require-effective-target int32plus } */
 
 extern void abort (void);
 
Index: gcc.c-torture/compile/pr43191.c
===================================================================
--- gcc.c-torture/compile/pr43191.c	(Revision 172597)
+++ gcc.c-torture/compile/pr43191.c	(Arbeitskopie)
@@ -1,4 +1,5 @@ 
-/* { dg-skip-if "Ints are 16 bits" { "pdp11-*-*" } { "*" } { "" } } */ 
+/* { dg-require-effective-target int32plus } */
+
 struct S0
 {
 };
Index: gcc.dg/pr42629.c
===================================================================
--- gcc.dg/pr42629.c	(Revision 172597)
+++ gcc.dg/pr42629.c	(Arbeitskopie)
@@ -2,6 +2,7 @@ 
    took debug insns into account.  */
 /* { dg-do compile } */
 /* { dg-options "-O1 -fsched-pressure -fschedule-insns -fcompare-debug" } */
+/* { dg-require-effective-target int32plus } */
 
 int lzo_adler32(int adler, char *buf)
 {
Index: gcc.dg/torture/pr43165.c
===================================================================
--- gcc.dg/torture/pr43165.c	(Revision 172597)
+++ gcc.dg/torture/pr43165.c	(Arbeitskopie)
@@ -1,5 +1,6 @@ 
 /* PR debug/43165 */
 /* { dg-options "-g" } */
+/* { dg-require-effective-target int32plus } */
 
 struct __attribute__((packed)) S
 {
Index: gcc.dg/torture/pr47228.c
===================================================================
--- gcc.dg/torture/pr47228.c	(Revision 172597)
+++ gcc.dg/torture/pr47228.c	(Arbeitskopie)
@@ -1,4 +1,5 @@ 
 /* { dg-do run } */
+/* { dg-require-effective-target int32plus } */
 
 struct S4
 {
Index: gcc.dg/tree-ssa/pr45144.c
===================================================================
--- gcc.dg/tree-ssa/pr45144.c	(Revision 172597)
+++ gcc.dg/tree-ssa/pr45144.c	(Arbeitskopie)
@@ -1,5 +1,6 @@ 
 /* { dg-do compile } */
 /* { dg-options "-O2 -fdump-tree-optimized" } */
+/* { dg-require-effective-target int32plus } */
 
 void baz (unsigned);
 
Index: gcc.dg/ipa/pr45644.c
===================================================================
--- gcc.dg/ipa/pr45644.c	(Revision 172597)
+++ gcc.dg/ipa/pr45644.c	(Arbeitskopie)
@@ -1,6 +1,7 @@ 
 /* Verify that we do not IPA-SRA bitfields.  */
 /* { dg-do run } */
 /* { dg-options "-O2"  } */
+/* { dg-require-effective-target int32plus } */
 
 extern void abort (void);