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.
vtst-2.diff
Description: Binary data
ChangeLog
Description: Binary data