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