Re: GCC 11 backport does not build (no "directly_supported_p") - was: Re: pr103523: Check for PLUS/MINUS support

2021-12-14 Thread Jakub Jelinek via Gcc-patches
On Tue, Dec 14, 2021 at 10:46:39AM +, Joel Hutton wrote: > > + if (ot_plus == unknown_optab > > + || ot_minus == unknown_optab > > + || optab_handler (ot_minus, TYPE_MODE (step_vectype)) == > > CODE_FOR_nothing > > + || optab_handler (ot_plus, TYPE_MODE (step_vectype)) == > > +

RE: GCC 11 backport does not build (no "directly_supported_p") - was: Re: pr103523: Check for PLUS/MINUS support

2021-12-14 Thread Joel Hutton via Gcc-patches
> + if (ot_plus == unknown_optab > + || ot_minus == unknown_optab > + || optab_handler (ot_minus, TYPE_MODE (step_vectype)) == > CODE_FOR_nothing > + || optab_handler (ot_plus, TYPE_MODE (step_vectype)) == > + CODE_FOR_nothing) > return false; > > Won't optab_handler just retu

Re: GCC 11 backport does not build (no "directly_supported_p") - was: Re: pr103523: Check for PLUS/MINUS support

2021-12-14 Thread Jakub Jelinek via Gcc-patches
On Tue, Dec 14, 2021 at 09:37:03AM +, Joel Hutton via Gcc-patches wrote: > Bootstrapped and regression tested on releases/gcc-11 on aarch64. > > Ok for 11? > > Previous commit broke build as it relied on directly_supported_p which > is not in 11. This reworks to avoid using directly_supported

RE: GCC 11 backport does not build (no "directly_supported_p") - was: Re: pr103523: Check for PLUS/MINUS support

2021-12-14 Thread Joel Hutton via Gcc-patches
ot;) - was: Re: pr103523: Check for PLUS/MINUS support My mistake, reworked patch. Tests are still running. From: Richard Biener mailto:richard.guent...@gmail.com>> Sent: 13 December 2021 14:47 To: gcc-patches@gcc.gnu.org<mailto:gcc-patches@gcc.gnu.org&g

Re: GCC 11 backport does not build (no "directly_supported_p") - was: Re: pr103523: Check for PLUS/MINUS support

2021-12-13 Thread Joel Hutton via Gcc-patches
(no "directly_supported_p") - was: Re: pr103523: Check for PLUS/MINUS support On December 13, 2021 3:27:50 PM GMT+01:00, Tobias Burnus wrote: >Hi Joel, > >your patch fails here with: > >../../repos/gcc-11-commit/gcc/tree-vect-loop.c:8000:8: error: >‘directly_supported

Re: GCC 11 backport does not build (no "directly_supported_p") - was: Re: pr103523: Check for PLUS/MINUS support

2021-12-13 Thread Richard Biener via Gcc-patches
t; >> From: Richard Sandiford >> Sent: 10 December 2021 10:22 >> To: Joel Hutton >> Cc: GCC Patches ; Richard Biener >> Subject: Re: pr103523: Check for PLUS/MINUS support >> >> Joel Hutton writes: >>> Hi

GCC 11 backport does not build (no "directly_supported_p") - was: Re: pr103523: Check for PLUS/MINUS support

2021-12-13 Thread Tobias Burnus
wrote: ok for backport to 11? From: Richard Sandiford Sent: 10 December 2021 10:22 To: Joel Hutton Cc: GCC Patches ; Richard Biener Subject: Re: pr103523: Check for PLUS/MINUS support Joel Hutton writes: Hi all, This is to address pr103523. bootstrapped and regre

Re: pr103523: Check for PLUS/MINUS support

2021-12-10 Thread Richard Sandiford via Gcc-patches
Joel Hutton writes: > ok for backport to 11? Yes, thanks. Richard

Re: pr103523: Check for PLUS/MINUS support

2021-12-10 Thread Joel Hutton via Gcc-patches
ok for backport to 11? From: Richard Sandiford Sent: 10 December 2021 10:22 To: Joel Hutton Cc: GCC Patches ; Richard Biener Subject: Re: pr103523: Check for PLUS/MINUS support Joel Hutton writes: > Hi all, > > This is to address pr103523. > > b

Re: pr103523: Check for PLUS/MINUS support

2021-12-10 Thread Richard Sandiford via Gcc-patches
Joel Hutton writes: > Hi all, > > This is to address pr103523. > > bootstrapped and regression tested on aarch64. > > Check for PLUS_EXPR/MINUS_EXPR support in vectorizable_induction. > PR103523 is an ICE on valid code: > > void d(float *a, float b, int c) { > float e; > for (; c; c--, e +