<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 2, 2020 at 8:35 PM Aditya Upadhyay <<a href="mailto:aadit0402@gmail.com">aadit0402@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">Hi Eshan,<br>
<br>
Thanks for pointing this out again. I would appreciate if you can send<br>
here a new v1 patch for the<br>
fenv methods. I can see the list of functions what you have left<br>
there, without any test.<br>
<a href="https://lists.rtems.org/pipermail/devel/2020-February/057669.html" rel="noreferrer" target="_blank">https://lists.rtems.org/pipermail/devel/2020-February/057669.html</a><br>
<br>
Could you please remove this?</blockquote><div>I have sent the patch for it <br></div><div><br></div><div><a href="https://lists.rtems.org/pipermail/devel/2020-February/057671.html">https://lists.rtems.org/pipermail/devel/2020-February/057671.html</a> </div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> I have not yet tested the code due to<br>
sparc-rtems-gdb and tar sim error.<br>
I will have to resolve this issue.<br></blockquote><div>Psxfenv01 tests won't work on sparc architecture since it doesn't have any implementation<br></div><div>of the floating-point environment in it yet.</div><div>i have checked them with riscv BSP <br></div><div><br></div><div> For each tests, you need to understand the behavior of that function<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
based on different parameters. I assume that<br>
you must have gone through the open group pages of these functions,<br>
there you will get to know the different<br>
outcomes. You will also have to look into some invalid set of inputs.<br></blockquote><div>ok  <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
If you are interested to contribute in POSIX Compliance, then I would<br>
suggest you to make yourself familiar with<br>
newlib-cygwin building process. You can also fix some warnings while<br>
building Newlib if you find one and send a patch to<br>
newlib mailing list. You may be spending most of the gsoc duration<br>
contributing to Newlib rather than RTEMS.<br>
<br></blockquote><div> thanks <br></div><div><br></div><div><br></div><div>--Eshan<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Regards,<br>
Aditya Upadhyay<br>
<br>
<br>
<br>
<br>
On Mon, Mar 2, 2020 at 1:26 AM Eshan Dhawan <<a href="mailto:eshandhawan51@gmail.com" target="_blank">eshandhawan51@gmail.com</a>> wrote:<br>
><br>
> hello everyone,<br>
> I am new here and I sent the patches:<br>
> 1. > <a href="https://lists.rtems.org/pipermail/devel/2020-February/057669.html" rel="noreferrer" target="_blank">https://lists.rtems.org/pipermail/devel/2020-February/057669.html</a><br>
> 2. > <a href="https://lists.rtems.org/pipermail/devel/2020-February/057671.html" rel="noreferrer" target="_blank">https://lists.rtems.org/pipermail/devel/2020-February/057671.html</a><br>
> 3. > <a href="https://lists.rtems.org/pipermail/devel/2020-February/057711.html" rel="noreferrer" target="_blank">https://lists.rtems.org/pipermail/devel/2020-February/057711.html</a><br>
> a few days ago.<br>
> for adding tests to psxfenv01.<br>
> but no one reviewed it.<br>
> so I ping it back<br>
> if someone could review it and give feedback.<br>
><br>
> -- Eshan<br>
><br>
> On Tue, Feb 25, 2020 at 11:15 PM Eshan Dhawan <<a href="mailto:eshandhawan51@gmail.com" target="_blank">eshandhawan51@gmail.com</a>> wrote:<br>
>><br>
>> added tests for fegetround and fesetround in psxtests / psxfenv01<br>
>><br>
>> fegetround fails to send the rounding direction due to soft float and returns to_nearest or zero  in the cases<br>
>> fesetround sends zero if the rounding direction is set and sends anything else if the function fails to set the rounding direction<br>
>><br>
>> can you check the patch for correctness<br>
>><br>
>> - Eshan<br>
>><br>
>> On Tue, Feb 25, 2020 at 8:26 PM Eshan dhawan <<a href="mailto:eshandhawan51@gmail.com" target="_blank">eshandhawan51@gmail.com</a>> wrote:<br>
>>><br>
>>> ---<br>
>>>  testsuites/psxtests/psxfenv01/init.c | 42 +++++++++++++++++++++++++++-<br>
>>>  1 file changed, 41 insertions(+), 1 deletion(-)<br>
>>><br>
>>> diff --git a/testsuites/psxtests/psxfenv01/init.c b/testsuites/psxtests/psxfenv01/init.c<br>
>>> index 05f3cdc880..4339139c58 100644<br>
>>> --- a/testsuites/psxtests/psxfenv01/init.c<br>
>>> +++ b/testsuites/psxtests/psxfenv01/init.c<br>
>>> @@ -106,7 +106,47 @@ rtems_task Init(rtems_task_argument ignored)<br>
>>>        printf("fesetexceptflag ==> 0x%x\n", r);<br>
>>>      rtems_test_assert(r == 0);<br>
>>><br>
>>> -<br>
>>> +<br>
>>> +    /*test for fegetround() and fesetround()*/<br>
>>> +    /*they have four main macros to be tested seperated by ifdef*/<br>
>>> +    /* since all the architectures dont support them */<br>
>>> +    /*the test cases gets and sets the rounding directions */<br>
>>> +#ifdef FE_TONEAREST<br>
>>> +<br>
>>> +    r=fegetround();<br>
>>> +    if(r)<br>
>>> +       printf("fegetround ==> 0x%x\n", r);<br>
>>> +    rtems_test_assert(r == FE_TONEAREST) ;<br>
>>> +#endif<br>
>>> +#ifdef FE_TOWARDZERO<br>
>>> +<br>
>>> +  r=fesetround(FE_TOWARDZERO);<br>
>>> +  if(r)<br>
>>> +       printf("fesetround ==> 0x%x\n", r);<br>
>>> +  rtems_test_assert(r == 0) ;<br>
>>> +  rtems_test_assert(fegetround() == FE_TOWARDZERO) ;<br>
>>> +#endif<br>
>>> +#ifdef FE_DOWNWARD<br>
>>> +<br>
>>> +  r=fesetround(FE_DOWNWARD);<br>
>>> +  if(r)<br>
>>> +       printf("fesetround ==> 0x%x\n", r);<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>
>>> +  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>
>>> +  rtems_test_assert(r == 0) ;<br>
>>> +#endif<br>
>>><br>
>>><br>
>>>  #ifdef FE_DIVBYZERO<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>