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  <a...@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

Reply via email to