Re: Fix 61441

2015-11-05 Thread Sujoy Saraswati
they should check if the particular > value is sNaN. I modified the code to check for sNaN specifically. The modified patch is below. Bootstrap and regression tests on x86_64-linux-gnu passed with changes done on trunk. Is this fine ? Regards, Sujoy 2015-11-04 Sujoy Saras

Re: Fix 61441

2015-11-05 Thread Sujoy Saraswati
Hi, > This is looking pretty close to ready to go in, but the patch also seems > to be doing several things at once and it might be easier to review if > split into separate logical pieces. For example (this is an illustration, > not necessarily saying it should be exactly this way): (1) Add >

Re: [Ping] Fix 61441

2015-10-26 Thread Sujoy Saraswati
This is a ping for the patch to fix 61441. Regards, Sujoy On Tue, Oct 13, 2015 at 4:16 PM, Sujoy Saraswati <ssarasw...@gmail.com> wrote: > Hi, > This is another modified version of the patch, incorporating the > previous comments. > > Bootstrap and regression test

Re: Fix 61441

2015-10-13 Thread Sujoy Saraswati
Hi, This is another modified version of the patch, incorporating the previous comments. Bootstrap and regression tests on x86_64-linux-gnu and aarch64-unknown-linux-gnu passed with changes done on trunk. Is this fine ? Regards, Sujoy 2015-10-13 Sujoy Saraswati <ssarasw...@gmail.

Re: Fix 61441

2015-09-16 Thread Sujoy Saraswati
Hi, >> I'll leave the correctness part of the patch to Joseph who knows FP >> arithmetic better than me, >> implementation-wise this is ok if you fix the REAL_CST sharing issue. Ok, will change this. > Changing fold_abs_const is wrong - abs of sNaN is sNaN, no exceptions > raised. Changing

Re: Fix 61441

2015-09-10 Thread Sujoy Saraswati
Hi, Here is a modified patch for this. The change this time is in fold-const.c and real.c. Bootstrap and regression tests on x86_64-linux-gnu and aarch64-unknown-linux-gnu passed with changes done on trunk. Is this fine ? Regards, Sujoy 2015-09-10 Sujoy Saraswati <ssarasw...@gmail.

Re: Fix 61441

2015-09-02 Thread Sujoy Saraswati
h. Let me know if you suggest alternate approach. Regards, Sujoy > Thanks, > Richard. > >> Regards, >> Sujoy >> >> 2015-09-01 Sujoy Saraswati <ssarasw...@gmail.com> >> >> PR tree-optimization/61441 >> * tree-

Re: Fix 61441

2015-09-02 Thread Sujoy Saraswati
it. Regards, Sujoy > > Richard. > >> Regards, >> Sujoy >> >>> Thanks, >>> Richard. >>> >>>> Regards, >>>> Sujoy >>>> >>>> 2015-09-01 Sujoy Saraswati <ssarasw...@gmail.com> >

Fix 61441

2015-09-01 Thread Sujoy Saraswati
The following patch fixes 61441. It converts sNaN to qNaN on folding when -fno-signaling-nans is used. Bootstrap and regression tests on x86_64-linux-gnu and aarch64-unknown-linux-gnu passed with changes done on trunk. Is this fix fine ? Regards, Sujoy 2015-09-01 Sujoy Saraswati <ssar