Re: [PATCH] Fix PR89437

2019-03-04 Thread Richard Biener
On Mon, Mar 4, 2019 at 2:39 PM Wilco Dijkstra wrote: > > Hi Richard, > > >On Thu, Feb 21, 2019 at 6:09 PM Wilco Dijkstra > >wrote: > >> > >> Hi Richard, > >> > >> >>Fix an issue with sinl (atanl (sqrtl (LDBL_MAX)) returning 0.0 > >> >>instead of 1.0 by using x < sqrtl (LDBL_MAX) in match.pd. > >

Re: [PATCH] Fix PR89437

2019-03-04 Thread Wilco Dijkstra
Hi Richard,   >On Thu, Feb 21, 2019 at 6:09 PM Wilco Dijkstra wrote: >> >> Hi Richard, >> >> >>Fix an issue with sinl (atanl (sqrtl (LDBL_MAX)) returning 0.0 >> >>instead of 1.0 by using x < sqrtl (LDBL_MAX) in match.pd. >> > >> > Wasn't that a intermediate problem with the mpfr exponent range li

Re: [PATCH] Fix PR89437

2019-02-22 Thread Richard Biener
On Thu, Feb 21, 2019 at 6:09 PM Wilco Dijkstra wrote: > > Hi Richard, > > >>Fix an issue with sinl (atanl (sqrtl (LDBL_MAX)) returning 0.0 > >>instead of 1.0 by using x < sqrtl (LDBL_MAX) in match.pd. > > > > Wasn't that a intermediate problem with the mpfr exponent range limiting? > > Please chec

Re: [PATCH] Fix PR89437

2019-02-21 Thread Jeff Law
On 2/21/19 8:55 AM, Wilco Dijkstra wrote: > Fix PR89437. Fix the sinatan-1.c testcase to not run without > a C99 target system. Use nextafterl for long double initialization. > > Fix an issue with sinl (atanl (sqrtl (LDBL_MAX)) returning 0.0 > instead of 1.0 by using x < sqrtl (LDBL_MAX) in match

Re: [PATCH] Fix PR89437

2019-02-21 Thread Wilco Dijkstra
Hi Richard, >>Fix an issue with sinl (atanl (sqrtl (LDBL_MAX)) returning 0.0 >>instead of 1.0 by using x < sqrtl (LDBL_MAX) in match.pd. > > Wasn't that a intermediate problem with the mpfr exponent range limiting? > Please check whether that's still needed. I tested it with trunk about an hour

Re: [PATCH] Fix PR89437

2019-02-21 Thread Giuliano Belinassi
I've just submitted a patch for this too :-P. Sorry about that. What is your nick in IRC, Wilco? On 02/21, Wilco Dijkstra wrote: > Fix PR89437. Fix the sinatan-1.c testcase to not run without > a C99 target system. Use nextafterl for long double initialization. > > Fix an issue with sinl (atan

Re: [PATCH] Fix PR89437

2019-02-21 Thread Richard Biener
On February 21, 2019 4:55:57 PM GMT+01:00, Wilco Dijkstra wrote: >Fix PR89437. Fix the sinatan-1.c testcase to not run without >a C99 target system. Use nextafterl for long double initialization. OK. >Fix an issue with sinl (atanl (sqrtl (LDBL_MAX)) returning 0.0 >instead of 1.0 by using x <