Re: [PATCH] gimple-range-op: Improve handling of sqrt ranges

2023-05-05 Thread Jakub Jelinek via Gcc-patches
On Fri, May 05, 2023 at 01:38:48PM +0200, Mikael Morin wrote: > I think there is something wrong with the handling of NAN and lower bounds > in op1_range. If the lhs has positive lower bound and can be nan, op1 > should have -inf lower bound regardless of lhs's lower bound value. Oops, you're rig

Re: [PATCH] gimple-range-op: Improve handling of sqrt ranges

2023-05-05 Thread Mikael Morin
Hello, Le 05/05/2023 à 10:00, Jakub Jelinek via Gcc-patches a écrit : Hi! The previous patch just added basic intrinsic ranges for sqrt ([-0.0, +Inf] +-NAN being the general result range of the function and [-0.0, +Inf] the general operand range if result isn't NAN etc.), the following patch in

Re: [PATCH] gimple-range-op: Improve handling of sqrt ranges

2023-05-05 Thread Aldy Hernandez via Gcc-patches
On Fri, May 5, 2023 at 11:14 AM Jakub Jelinek wrote: > > On Fri, May 05, 2023 at 11:06:31AM +0200, Aldy Hernandez wrote: > > > +/* Compute FUNC (ARG) where FUNC is a mpfr function. If RES_LOW is > > > non-NULL, > > > + set it to low bound of possible range if the function is expected to > > >

Re: [PATCH] gimple-range-op: Improve handling of sqrt ranges

2023-05-05 Thread Jakub Jelinek via Gcc-patches
On Fri, May 05, 2023 at 11:06:31AM +0200, Aldy Hernandez wrote: > > +/* Compute FUNC (ARG) where FUNC is a mpfr function. If RES_LOW is > > non-NULL, > > + set it to low bound of possible range if the function is expected to > > have > > + ULPS precision and similarly if RES_HIGH is non-NULL

Re: [PATCH] gimple-range-op: Improve handling of sqrt ranges

2023-05-05 Thread Aldy Hernandez via Gcc-patches
On 5/5/23 10:00, Jakub Jelinek wrote: Hi! The previous patch just added basic intrinsic ranges for sqrt ([-0.0, +Inf] +-NAN being the general result range of the function and [-0.0, +Inf] the general operand range if result isn't NAN etc.), the following patch intersects those ranges with par

[PATCH] gimple-range-op: Improve handling of sqrt ranges

2023-05-05 Thread Jakub Jelinek via Gcc-patches
Hi! The previous patch just added basic intrinsic ranges for sqrt ([-0.0, +Inf] +-NAN being the general result range of the function and [-0.0, +Inf] the general operand range if result isn't NAN etc.), the following patch intersects those ranges with particular range computed from argument or res