Re: [PATCH] [PR tree-optimization/107394] Canonicalize global franges as they are read back.

2022-10-28 Thread Aldy Hernandez via Gcc-patches
On Fri, Oct 28, 2022, 08:49 Richard Biener wrote: > On Fri, Oct 28, 2022 at 12:45 AM Jeff Law wrote: > > > > > > On 10/25/22 15:01, Aldy Hernandez via Gcc-patches wrote: > > > [Richi/Jakub/FP experts, does this sound like the right solution, or > am I > > > missing some subtle IPA/inlining issue

Re: [PATCH] [PR tree-optimization/107394] Canonicalize global franges as they are read back.

2022-10-27 Thread Richard Biener via Gcc-patches
On Fri, Oct 28, 2022 at 8:48 AM Richard Biener wrote: > > On Fri, Oct 28, 2022 at 12:45 AM Jeff Law wrote: > > > > > > On 10/25/22 15:01, Aldy Hernandez via Gcc-patches wrote: > > > [Richi/Jakub/FP experts, does this sound like the right solution, or am I > > > missing some subtle IPA/inlining is

Re: [PATCH] [PR tree-optimization/107394] Canonicalize global franges as they are read back.

2022-10-27 Thread Richard Biener via Gcc-patches
On Fri, Oct 28, 2022 at 12:45 AM Jeff Law wrote: > > > On 10/25/22 15:01, Aldy Hernandez via Gcc-patches wrote: > > [Richi/Jakub/FP experts, does this sound like the right solution, or am I > > missing some subtle IPA/inlining issue?] > > > > The problem here is that we're inlining a global range

Re: [PATCH] [PR tree-optimization/107394] Canonicalize global franges as they are read back.

2022-10-27 Thread Jeff Law via Gcc-patches
On 10/25/22 15:01, Aldy Hernandez via Gcc-patches wrote: [Richi/Jakub/FP experts, does this sound like the right solution, or am I missing some subtle IPA/inlining issue?] The problem here is that we're inlining a global range with NANs into a function that has been tagged with __attribute__((

Re: [PATCH] [PR tree-optimization/107394] Canonicalize global franges as they are read back.

2022-10-26 Thread Aldy Hernandez via Gcc-patches
OR_NANS, but we must > + // make sure to set the NAN sign if known. > + if (HONOR_NANS (type) && (m_pos_nan ^ m_neg_nan) == 1) > +r.update_nan (m_neg_nan); > } > > bool > -- > 2.37.3 > From 89eaccbc4b1e5ff7fea2eb4e3fb5870c6ca4160b Mon Sep 17 00:00:00 2001 Fro

[PATCH] [PR tree-optimization/107394] Canonicalize global franges as they are read back.

2022-10-25 Thread Aldy Hernandez via Gcc-patches
[Richi/Jakub/FP experts, does this sound like the right solution, or am I missing some subtle IPA/inlining issue?] The problem here is that we're inlining a global range with NANs into a function that has been tagged with __attribute__((optimize ("-ffinite-math-only"))). As the global range is co