I have added the changes as told by Aditya 
I would like to know is any other changes are required to be made to make the 
patch usable 

Thanks 
Eshan 
> On 21-Mar-2020, at 8:20 PM, Eshan dhawan <eshandhawa...@gmail.com> wrote:
> 
> ---
> testsuites/psxtests/psxfenv01/init.c | 89 ++++++++++++++++++++++------
> 1 file changed, 71 insertions(+), 18 deletions(-)
> 
> diff --git a/testsuites/psxtests/psxfenv01/init.c 
> b/testsuites/psxtests/psxfenv01/init.c
> index cdb0fa596e..0ea820c523 100644
> --- a/testsuites/psxtests/psxfenv01/init.c
> +++ b/testsuites/psxtests/psxfenv01/init.c
> @@ -46,11 +46,12 @@
> #include <string.h>
> #include <rtems/test.h>
> #include <tmacros.h>
> +#include <float.h>
> 
> const char rtems_test_name[] ="PSXFENV 01";
> 
> /* forward declarations to avoid warnings */
> -rtems_task Init(rtems_task_argument ignored);
> +rtems_task Init( rtems_task_argument ignored );
> 
> /* Test Function Begins */
> rtems_task Init(rtems_task_argument ignored)
> @@ -62,28 +63,31 @@ rtems_task Init(rtems_task_argument ignored)
> 
>  /*
>   * 'FE_ALL_EXCEPT' will be defined only when 'feclearexcept()',
> -   * 'fegetexceptflag()', 'feraiseexcept()', 'fesetexceptflag()' and
> -   * 'fetestexcept()' functions are supported by the architecture.
> +   * fegetexceptflag() , feraiseexcept(), fesetexceptflag() and
> +   * fetestexcept() functions are supported by the architecture.
>   * Hence their testcases can be wrapped under #ifdef and #endif.
>   */
>  #ifdef FE_ALL_EXCEPT /* floating-point exceptions */
>    puts( "fesetenv(FE_DFL_ENV)." );
> -    r = fesetenv(FE_DFL_ENV);
> -    if (r)
> +    r = fesetenv ( FE_DFL_ENV );
> +    if ( r ) {
>      printf("fesetenv ==> %d\n", r);
> -    rtems_test_assert( r == 0 );
> +    }
> +    rtems_test_assert ( r == 0 );
> 
> -    /* Test 'feclearexcept()' and 'fetestexcept()' in one go. */
> -    puts( "feclearexcept(FE_ALL_EXCEPT)." );
> -    r = feclearexcept(FE_ALL_EXCEPT);
> -    if (r)
> +    /* Test feclearexcept() and fetestexcept() in one go. */
> +    puts( "feclearexcept(FE_ALL_EXCEPT)" );
> +    r = feclearexcept ( FE_ALL_EXCEPT );
> +    if ( r ) {
>      printf("feclearexcept ==> 0x%x\n", r);
> -    rtems_test_assert( r == 0 );
> +    }    
> +    rtems_test_assert ( r == 0 );
> 
> -    r = fetestexcept( FE_ALL_EXCEPT );
> -    if (r)
> +    r = fetestexcept ( FE_ALL_EXCEPT );
> +    if ( r ) {
>      printf("fetestexcept ==> 0x%x\n", r);
> -    rtems_test_assert( r == 0 );
> +    }
> +    rtems_test_assert ( r == 0 );
> 
>    /* Test 'FE_DIVBYZERO' */
>    puts( "Divide by zero and confirm fetestexcept()" );
> @@ -91,11 +95,60 @@ rtems_task Init(rtems_task_argument ignored)
>    b = 1.0;
>    c = b/a;
>    (void) c;
> -
> -    fegetexceptflag(&excepts,FE_ALL_EXCEPT);
> -
> +    /* Test fegetexceptflag() and fesetexceptflag() */
> +    r = fegetexceptflag ( &excepts, FE_ALL_EXCEPT );
> +    if ( r ) {
> +      printf("fegetexceptflag ==> 0x%x\n", r);
> +    }
> +    rtems_test_assert ( r == 0 );
> +
> +    r = fesetexceptflag ( &excepts, FE_ALL_EXCEPT );
> +    if ( r ) {
> +      printf("fesetexceptflag ==> 0x%x\n", r);
> +    }
> +    rtems_test_assert ( r == 0 );
> +
> +    /* Test for fegetround() and fesetround()
> +     * They have four main macros to be tested separated by ifdef
> +     * Since not all architectures support them 
> +     * The test case gets and sets the rounding directions */
> +#ifdef FE_TONEAREST
> +    rtems_test_assert ( fegetround() == FE_TONEAREST );   
> +#endif
> +#ifdef FE_TOWARDZERO    
> +  r = fesetround ( FE_TOWARDZERO );
> +  if ( r ) {
> +      printf("fesetround ==> 0x%x\n", r);     
> +    } 
> +  rtems_test_assert ( r == 0 );
> +  rtems_test_assert ( fegetround() == FE_TOWARDZERO );
> +#endif
> +#ifdef FE_DOWNWARD
> +  r = fesetround ( FE_DOWNWARD );
> +  if ( r ) {
> +      printf("fesetround ==> 0x%x\n", r);     
> +    } 
> +  rtems_test_assert ( r == 0 );
> +  rtems_test_assert ( fegetround() == FE_DOWNWARD );
> +#endif
> +#ifdef FE_UPWARD
> +  r = fesetround ( FE_UPWARD );
> +  if ( r ) {
> +      printf("fesetround ==> 0x%x\n", r);     
> +    }   
> +  rtems_test_assert ( r == 0 );
> +  rtems_test_assert ( fegetround() == FE_UPWARD );
> +#endif
> +#ifdef FE_TONEAREST
> +  r = fesetround ( FE_TONEAREST );
> +  if ( r ) {
> +      printf("fesetround ==> 0x%x\n", r);
> +    }     
> +  rtems_test_assert ( r == 0 );
> +#endif
> +  
> #ifdef FE_DIVBYZERO
> -    r = feraiseexcept(FE_DIVBYZERO);
> +    r = feraiseexcept ( FE_DIVBYZERO ) ;
>    rtems_test_assert( fetestexcept( FE_DIVBYZERO ) );
> #endif
> 
> -- 
> 2.17.1
_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel

Reply via email to