Re: [PATCH] Rewrite NAN and sign handling in frange

2022-11-02 Thread Aldy Hernandez via Gcc-patches
On 9/27/22 15:00, Mikael Morin wrote: Hello, Le 16/09/2022 à 15:26, Aldy Hernandez via Gcc-patches a écrit : diff --git a/gcc/value-range.cc b/gcc/value-range.cc index d759fcf178c..55a216efd8b 100644 --- a/gcc/value-range.cc +++ b/gcc/value-range.cc @@ -617,21 +602,24 @@ frange::contains_p (t

Re: [PATCH] Rewrite NAN and sign handling in frange

2022-09-27 Thread Mikael Morin
Hello, Le 16/09/2022 à 15:26, Aldy Hernandez via Gcc-patches a écrit : diff --git a/gcc/value-range.cc b/gcc/value-range.cc index d759fcf178c..55a216efd8b 100644 --- a/gcc/value-range.cc +++ b/gcc/value-range.cc @@ -617,21 +602,24 @@ frange::contains_p (tree cst) const if (varying_p ())

Re: [PATCH] Rewrite NAN and sign handling in frange

2022-09-18 Thread Aldy Hernandez via Gcc-patches
Pushed. We can address any further changes as follow-ups. Thanks. Aldy On Fri, Sep 16, 2022 at 3:26 PM Aldy Hernandez wrote: > > On Fri, Sep 16, 2022 at 10:33 AM Richard Sandiford > wrote: > > > > Aldy Hernandez via Gcc-patches writes: > > > On Thu, Sep 15, 2022 at 9:06 AM Richard Biener > >

Re: [PATCH] Rewrite NAN and sign handling in frange

2022-09-16 Thread Aldy Hernandez via Gcc-patches
On Fri, Sep 16, 2022 at 10:33 AM Richard Sandiford wrote: > > Aldy Hernandez via Gcc-patches writes: > > On Thu, Sep 15, 2022 at 9:06 AM Richard Biener > > wrote: > >> > >> On Thu, Sep 15, 2022 at 7:41 AM Aldy Hernandez wrote: > >> > > >> > Hi Richard. Hi all. > >> > > >> > The attatched patch

Re: [PATCH] Rewrite NAN and sign handling in frange

2022-09-16 Thread Richard Sandiford via Gcc-patches
Aldy Hernandez via Gcc-patches writes: > On Thu, Sep 15, 2022 at 9:06 AM Richard Biener > wrote: >> >> On Thu, Sep 15, 2022 at 7:41 AM Aldy Hernandez wrote: >> > >> > Hi Richard. Hi all. >> > >> > The attatched patch rewrites the NAN and sign handling, dropping both >> > tristates in favor of a

Re: [PATCH] Rewrite NAN and sign handling in frange

2022-09-15 Thread Aldy Hernandez via Gcc-patches
On Thu, Sep 15, 2022 at 9:06 AM Richard Biener wrote: > > On Thu, Sep 15, 2022 at 7:41 AM Aldy Hernandez wrote: > > > > Hi Richard. Hi all. > > > > The attatched patch rewrites the NAN and sign handling, dropping both > > tristates in favor of a pair of boolean flags for NANs, and nothing at > >

Re: [PATCH] Rewrite NAN and sign handling in frange

2022-09-15 Thread Richard Biener via Gcc-patches
On Thu, Sep 15, 2022 at 7:41 AM Aldy Hernandez wrote: > > Hi Richard. Hi all. > > The attatched patch rewrites the NAN and sign handling, dropping both > tristates in favor of a pair of boolean flags for NANs, and nothing at > all for signs. The signs are tracked in the range itself, so now it's