Re: [committed] [testsuite] Fix fp-int-convert-timode-1.c testism.

[Tamar Christina]

Hi Joseph,

> > Or do you want me to send them separately?
> 
> I think it's best to fix the test now not to have the #ifdef, then if you 
> have execution failures those can be addressed separately.  (If you want 
> to avoid the test FAILing before then, an XFAIL with a comment referencing 
> an open bug in Bugzilla would be appropriate, not #ifdef that makes the 
> test spuriously PASS.)
> 

Fair enough, found the issue and it wasn't with the test. I've attached the
new patch.

Ok for trunk?

Thanks,
Tamar

> -- 
> Joseph S. Myers
> jos...@codesourcery.com

-- 

diff --git a/gcc/testsuite/gcc.dg/torture/fp-int-convert-timode-1.c b/gcc/testsuite/gcc.dg/torture/fp-int-convert-timode-1.c
index bf7f3cedb294cc834437593dae3507005f0f6b56..971a5985357ce50e187d3dea2660804c8055e141 100644
--- a/gcc/testsuite/gcc.dg/torture/fp-int-convert-timode-1.c
+++ b/gcc/testsuite/gcc.dg/torture/fp-int-convert-timode-1.c
@@ -2,27 +2,22 @@
    float.  */
 /* { dg-do run } */
 /* { dg-require-effective-target int128 } */
-/* { dg-require-effective-target fenv } */
 /* { dg-options "-frounding-math" } */
 
-#include <fenv.h>
 #include <stdlib.h>
 
 int
 main (void)
 {
-#ifdef FE_TONEAREST
   volatile unsigned long long h = 0x8000000000000000LL;
   volatile unsigned long long l = 0xdLL;
   volatile unsigned __int128 u128 = (((unsigned __int128) h) << 64) | l;
   volatile __int128 s128 = u128;
-  fesetround (FE_TONEAREST);
   float fs = s128;
   if (fs != -0x1p+127)
     abort ();
   double ds = s128;
   if (ds != -0x1p+127)
     abort ();
-#endif
   exit (0);
 }