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
amp; REAL_VALUE_ISNAN when 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 Suj
This is a ping for the patch to fix 61441.
Regards,
Sujoy
On Tue, Oct 13, 2015 at 4:16 PM, Sujoy Saraswati wrote:
> Hi,
> This is another modified version of the patch, incorporating the
> previous comments.
>
> Bootstrap and regression tests on x86_64-linux-gnu and
> aarch64
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
PR tree-optimization
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 real
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
PR tree
it.
Regards,
Sujoy
>
> Richard.
>
>> Regards,
>> Sujoy
>>
>>> Thanks,
>>> Richard.
>>>
>>>> Regards,
>>>> Sujoy
>>>>
>>>> 2015-09-01 Sujoy Saraswati
>>>>
>>>> PR t
start with.
Let me know if you suggest alternate approach.
Regards,
Sujoy
> Thanks,
> Richard.
>
>> Regards,
>> Sujoy
>>
>> 2015-09-01 Sujoy Saraswati
>>
>> PR tree-optimization/61441
>> * tree-ssa-ccp.c (convert_snan
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
PR
