On Mon, 10 May 2021 at 19:55, Richard Earnshaw
<richard.earns...@foss.arm.com> wrote:
>
>
>
> On 06/05/2021 01:14, Prathamesh Kulkarni via Gcc-patches wrote:
> > Hi,
> > The attached patch replaces __builtin_neon_vtst* (a, b) with (a & b) != 0.
> > Bootstrapped and tested on arm-linux-gnueabihf and cross-tested on arm*-*-*.
> > OK to commit ?
> >
> > Thanks,
> > Prathamesh
> >
>
> You're missing the ChangeLog details.
>
> Also, if you're removing these, do we still need the neon_vtst<mode>
> patterns in neon.md?  They generate unspecs, so if they're no-longer
> needed for these expansions, they can likely be dropped entirely.
Hi Richard,
Thanks for the suggestions.
Does the attached patch look OK if bootstrap+test passes ?

Thanks,
Prathamesh
>
> R.

Attachment: vtst-2.diff
Description: Binary data

Attachment: ChangeLog
Description: Binary data

Reply via email to