<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Mar 21, 2020 at 12:19 AM Eshan Dhawan <<a href="mailto:eshandhawan51@gmail.com">eshandhawan51@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Hello Everyone,</div><div>I had sent this patch a few days ago to add tests of fenv.h to psxtests.<br></div><div>If you all could look into it and tell if it could be merged or requires any more changes.<br><br></div></div></blockquote><div>link to patch :<strong><strong><span style="font-size:10pt;font-family:Verdana;color:rgb(0,0,0);background-color:transparent;font-weight:400;font-style:normal;font-variant:normal;text-decoration:none;vertical-align:baseline;white-space:pre-wrap" id="gmail-docs-internal-guid-f33ab7d8-7fff-69ac-1528-1c9e2fd73525"> </span><a href="https://lists.rtems.org/pipermail/devel/2020-March/058063.html" style="text-decoration:none"><span style="font-size:10pt;font-family:Verdana;color:rgb(0,0,128);background-color:transparent;font-weight:400;font-style:normal;font-variant:normal;text-decoration:underline;vertical-align:baseline;white-space:pre-wrap">https://lists.rtems.org/pipermail/devel/2020-March/058063.html</span></a></strong></strong> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div></div><div>thanks,</div><div>Eshan Dhawan  <br></div><div> <br> </div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Mar 8, 2020 at 10:32 PM Eshan Dhawan <<a href="mailto:eshandhawan51@gmail.com" target="_blank">eshandhawan51@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div><div>I have compiled as well as executed it for RISC V architecture function Fegetround() returns error for all rounding directions other the default "FETONEAREST" I asked about it on devel to which dr Joel told that it is due to soft-float. <br></div></div>So I have added its test case but they won't work on emulators. <br></div><div><br></div><div><br></div><div>thanks <br></div><div>-Eshan <br></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Mar 8, 2020 at 7:55 PM Gedare Bloom <<a href="mailto:gedare@rtems.org" target="_blank">gedare@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Eshan,<br>
<br>
This is looking good. Did you compile it? Execute it?  How was the output?<br>
<br>
On Sat, Mar 7, 2020 at 6:51 AM Eshan dhawan <<a href="mailto:eshandhawan51@gmail.com" target="_blank">eshandhawan51@gmail.com</a>> wrote:<br>
><br>
> ---<br>
>  testsuites/psxtests/psxfenv01/init.c | 89 ++++++++++++++++++++++------<br>
>  1 file changed, 71 insertions(+), 18 deletions(-)<br>
><br>
> diff --git a/testsuites/psxtests/psxfenv01/init.c b/testsuites/psxtests/psxfenv01/init.c<br>
> index cdb0fa596e..6d71463465 100644<br>
> --- a/testsuites/psxtests/psxfenv01/init.c<br>
> +++ b/testsuites/psxtests/psxfenv01/init.c<br>
> @@ -46,11 +46,12 @@<br>
>  #include <string.h><br>
>  #include <rtems/test.h><br>
>  #include <tmacros.h><br>
> +#include <float.h><br>
><br>
>  const char rtems_test_name[] = "PSXFENV 01";<br>
><br>
>  /* forward declarations to avoid warnings */<br>
> -rtems_task Init(rtems_task_argument ignored);<br>
> +rtems_task Init( rtems_task_argument ignored );<br>
><br>
>  /* Test Function Begins */<br>
>  rtems_task Init(rtems_task_argument ignored)<br>
> @@ -62,28 +63,31 @@ rtems_task Init(rtems_task_argument ignored)<br>
><br>
>    /*<br>
>     * 'FE_ALL_EXCEPT' will be defined only when 'feclearexcept()',<br>
> -   * 'fegetexceptflag()', 'feraiseexcept()', 'fesetexceptflag()' and<br>
> -   * 'fetestexcept()' functions are supported by the architecture.<br>
> +   * fegetexceptflag() , feraiseexcept(), fesetexceptflag() and<br>
> +   * fetestexcept() functions are supported by the architecture.<br>
>     * Hence their testcases can be wrapped under #ifdef and #endif.<br>
>     */<br>
>    #ifdef FE_ALL_EXCEPT /* floating-point exceptions */<br>
>      puts( "fesetenv(FE_DFL_ENV)." );<br>
> -    r = fesetenv(FE_DFL_ENV);<br>
> -    if (r)<br>
> +    r = fesetenv ( FE_DFL_ENV );<br>
> +    if ( r ) {<br>
>        printf("fesetenv ==> %d\n", r);<br>
> -    rtems_test_assert( r == 0 );<br>
> +    }<br>
> +    rtems_test_assert ( r == 0 );<br>
><br>
> -    /* Test 'feclearexcept()' and 'fetestexcept()' in one go. */<br>
> -    puts( "feclearexcept(FE_ALL_EXCEPT)." );<br>
> -    r = feclearexcept(FE_ALL_EXCEPT);<br>
> -    if (r)<br>
> +    /* Test feclearexcept() and fetestexcept() in one go. */<br>
> +    puts( "feclearexcept(FE_ALL_EXCEPT)" );<br>
> +    r = feclearexcept ( FE_ALL_EXCEPT );<br>
> +    if ( r ) {<br>
>        printf("feclearexcept ==> 0x%x\n", r);<br>
> -    rtems_test_assert( r == 0 );<br>
> +    }<br>
> +    rtems_test_assert ( r == 0 );<br>
><br>
> -    r = fetestexcept( FE_ALL_EXCEPT );<br>
> -    if (r)<br>
> +    r = fetestexcept ( FE_ALL_EXCEPT );<br>
> +    if ( r ) {<br>
>        printf("fetestexcept ==> 0x%x\n", r);<br>
> -    rtems_test_assert( r == 0 );<br>
> +    }<br>
> +    rtems_test_assert ( r == 0 );<br>
><br>
>      /* Test 'FE_DIVBYZERO' */<br>
>      puts( "Divide by zero and confirm fetestexcept()" );<br>
> @@ -91,11 +95,60 @@ rtems_task Init(rtems_task_argument ignored)<br>
>      b = 1.0;<br>
>      c = b/a;<br>
>      (void) c;<br>
> -<br>
> -    fegetexceptflag(&excepts,FE_ALL_EXCEPT);<br>
> -<br>
> +    /* Test fegetexceptflag() and fesetexceptflag() */<br>
> +    r = fegetexceptflag ( &excepts, FE_ALL_EXCEPT );<br>
> +    if ( r ) {<br>
> +      printf("fegetexceptflag ==> 0x%x\n", r);<br>
> +    }<br>
> +    rtems_test_assert ( r == 0 );<br>
> +<br>
> +    r = fesetexceptflag ( &excepts, FE_ALL_EXCEPT );<br>
> +    if ( r ) {<br>
> +      printf("fesetexceptflag ==> 0x%x\n", r);<br>
> +    }<br>
> +    rtems_test_assert ( r == 0 );<br>
> +<br>
> +    /* Test for fegetround() and fesetround()<br>
> +     * They have four main macros to be tested separated by ifdef<br>
> +     * Since not all architectures support them<br>
> +     * The test case gets and sets the rounding directions */<br>
> +#ifdef FE_TONEAREST<br>
> +    rtems_test_assert ( fegetround() == FE_TONEAREST ) ;<br>
> +#endif<br>
> +#ifdef FE_TOWARDZERO<br>
> +  r = fesetround ( FE_TOWARDZERO ) ;<br>
> +  if ( r ) {<br>
> +      printf("fesetround ==> 0x%x\n", r);<br>
> +    }<br>
> +  rtems_test_assert ( r == 0 ) ;<br>
> +  rtems_test_assert ( fegetround() == FE_TOWARDZERO ) ;<br>
> +#endif<br>
> +#ifdef FE_DOWNWARD<br>
> +  r = fesetround ( FE_DOWNWARD );<br>
> +  if ( r ) {<br>
> +      printf("fesetround ==> 0x%x\n", r);<br>
> +    }<br>
> +  rtems_test_assert ( r == 0 ) ;<br>
> +  rtems_test_assert ( fegetround() == FE_DOWNWARD ) ;<br>
> +#endif<br>
> +#ifdef FE_UPWARD<br>
> +  r = fesetround ( FE_UPWARD );<br>
> +  if ( r ) {<br>
> +      printf("fesetround ==> 0x%x\n", r);<br>
> +    }<br>
> +  rtems_test_assert ( r == 0 ) ;<br>
> +  rtems_test_assert ( fegetround() == FE_UPWARD ) ;<br>
> +#endif<br>
> +#ifdef FE_TONEAREST<br>
> +  r = fesetround ( FE_TONEAREST );<br>
> +  if ( r ) {<br>
> +      printf("fesetround ==> 0x%x\n", r);<br>
> +    }<br>
> +  rtems_test_assert ( r == 0 ) ;<br>
> +#endif<br>
> +<br>
>  #ifdef FE_DIVBYZERO<br>
> -    r = feraiseexcept(FE_DIVBYZERO);<br>
> +    r = feraiseexcept ( FE_DIVBYZERO ) ;<br>
>      rtems_test_assert( fetestexcept( FE_DIVBYZERO ) );<br>
>  #endif<br>
><br>
> --<br>
> 2.17.1<br>
><br>
> _______________________________________________<br>
> devel mailing list<br>
> <a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div></div>
</blockquote></div>
</blockquote></div></div>