Re: [PATCH v9] rtl: builtins: (not just) rs6000: Add builtins for fegetround, feclearexcept and feraiseexcept [PR94193]

2022-01-25 Thread Jakub Jelinek via Gcc-patches
On Tue, Jan 25, 2022 at 01:28:29PM -0300, Raoni Fassina Firmino wrote: > Below is a patch to do just that. In preliminary tests it seems to work. > What do you think aboud it Jakub? Ok for trunk. > > These days the usual way of doing this is through > > maybe_expand_insn and

Re: [PATCH v9] rtl: builtins: (not just) rs6000: Add builtins for fegetround, feclearexcept and feraiseexcept [PR94193]

2022-01-25 Thread Raoni Fassina Firmino via Gcc-patches
On Mon, Jan 24, 2022 at 11:35:47PM +0100, AL gcc-patches wrote: > On Mon, Jan 24, 2022 at 06:24:11PM -0300, Raoni Fassina Firmino wrote: > > On Mon, Jan 24, 2022 at 02:29:39PM -0600, Bill Schmidt wrote: > > > Adding the patch author for his information. > > > > Thanks Bill. > > > > > On 1/24/22

Re: [PATCH v9] rtl: builtins: (not just) rs6000: Add builtins for fegetround, feclearexcept and feraiseexcept [PR94193]

2022-01-24 Thread Jakub Jelinek via Gcc-patches
On Mon, Jan 24, 2022 at 06:24:11PM -0300, Raoni Fassina Firmino wrote: > On Mon, Jan 24, 2022 at 02:29:39PM -0600, Bill Schmidt wrote: > > Adding the patch author for his information. > > Thanks Bill. > > > On 1/24/22 2:26 PM, Jakub Jelinek via Gcc-patches wrote: > > >

Re: [PATCH v9] rtl: builtins: (not just) rs6000: Add builtins for fegetround, feclearexcept and feraiseexcept [PR94193]

2022-01-24 Thread Raoni Fassina Firmino via Gcc-patches
On Mon, Jan 24, 2022 at 02:29:39PM -0600, Bill Schmidt wrote: > Adding the patch author for his information. Thanks Bill. > On 1/24/22 2:26 PM, Jakub Jelinek via Gcc-patches wrote: > > expand_builtin_feclear_feraise_except doesn't check if op0 matches > > the predicate of operands[1], the

Re: [PATCH v9] rtl: builtins: (not just) rs6000: Add builtins for fegetround, feclearexcept and feraiseexcept [PR94193]

2022-01-24 Thread Bill Schmidt via Gcc-patches
Adding the patch author for his information. Thanks, Bill On 1/24/22 2:26 PM, Jakub Jelinek via Gcc-patches wrote: > On Mon, Jan 24, 2022 at 08:55:37AM -0600, Segher Boessenkool wrote: >> Hi! >> >> On Thu, Jan 13, 2022 at 02:08:53PM -0300, Raoni Fassina Firmino wrote: >>> Changes since v8[8]:

Re: [PATCH v9] rtl: builtins: (not just) rs6000: Add builtins for fegetround, feclearexcept and feraiseexcept [PR94193]

2022-01-24 Thread Jakub Jelinek via Gcc-patches
On Mon, Jan 24, 2022 at 08:55:37AM -0600, Segher Boessenkool wrote: > Hi! > > On Thu, Jan 13, 2022 at 02:08:53PM -0300, Raoni Fassina Firmino wrote: > > Changes since v8[8]: > > - Refactored and expanded builtin-feclearexcept-feraiseexcept-2.c > > testcase: > > + Use a macro to avoid

Re: [PATCH v9] rtl: builtins: (not just) rs6000: Add builtins for fegetround, feclearexcept and feraiseexcept [PR94193]

2022-01-24 Thread Segher Boessenkool
Hi! On Thu, Jan 13, 2022 at 02:08:53PM -0300, Raoni Fassina Firmino wrote: > Changes since v8[8]: > - Refactored and expanded builtin-feclearexcept-feraiseexcept-2.c > testcase: > + Use a macro to avoid extended repetition of the core test code. > + Expanded the test code to check

[PATCH v9] rtl: builtins: (not just) rs6000: Add builtins for fegetround, feclearexcept and feraiseexcept [PR94193]

2022-01-13 Thread Raoni Fassina Firmino via Gcc-patches
Changes since v8[8]: - Refactored and expanded builtin-feclearexcept-feraiseexcept-2.c testcase: + Use a macro to avoid extended repetition of the core test code. + Expanded the test code to check builtins return code. + Added more tests to test all valid (standard) exceptions