PATCH] skip gcc.dg/torture/fp-int-convert-float128*.c on cygwin
David Billinghurst
dbcygwin@gmail.com
Sat May 16 01:24:00 GMT 2009
David Billinghurst wrote:
> Tests gcc.dg/torture/fp-int-convert-float128-timode.c and
> gcc.dg/torture/fp-int-convert-float128.c fail on a number of platforms
> on gcc-4.3, 4.4 and trunk.
>
> For some examples, see:
> - http://gcc.gnu.org/ml/gcc-testresults/2009-05/msg00552.html
> - http://gcc.gnu.org/ml/gcc-testresults/2009-05/msg00315.html
> - http://gcc.gnu.org/ml/gcc-testresults/2009-04/msg02417.html
>
> At present the tests are run for
> { dg-do run { target i?86-*-* x86_64-*-* ia64-*-* } }
>
> 128 bit floats aren't supported on cygwin, and I expect the same is true
> elsewhere. This patch just skips the test for cygwin, and will be
> simple to extend to other targets as required. The alternative approach
> of modifying the dg-do run target became messy.
>
> OK for trunk, and 4.3 and 4.4 after an interval?
>
>
> 2009-05-16 David Billinghurst <billingd@gcc.gnu.org>
>
> * gcc.dg/torture/fp-int-convert-float128.c fail: Skip on cygwin
> * gcc.dg/torture/fp-int-convert-float128-timode.c: Likewise
>
> Index: fp-int-convert-float128-timode.c
> ===================================================================
> --- fp-int-convert-float128-timode.c (revision 147328)
> +++ fp-int-convert-float128-timode.c (working copy)
> @@ -1,6 +1,7 @@
> /* Test floating-point conversions. __float128 type with TImode. */
> /* Origin: Joseph Myers <joseph@codesourcery.com> */
> /* { dg-do run { target i?86-*-* x86_64-*-* ia64-*-* } } */
> +/* { dg-skip-if "" { *-*-cygwin* } } */
> /* { dg-options "" } */
>
> #include "fp-int-convert.h"
> Index: fp-int-convert-float128.c
> ===================================================================
> --- fp-int-convert-float128.c (revision 147328)
> +++ fp-int-convert-float128.c (working copy)
> @@ -1,6 +1,7 @@
> /* Test floating-point conversions. __float128 type. */
> /* Origin: Joseph Myers <joseph@codesourcery.com> */
> /* { dg-do run { target i?86-*-* x86_64-*-* ia64-*-* } } */
> +/* { dg-skip-if "" { *-*-cygwin* } } */
> /* { dg-options "" } */
>
> #include "fp-int-convert.h"
>
This is not the patch I tested - the skip-if syntax is wrong. I will
find the correct one and resubmit.
More information about the Gcc-patches
mailing list