Hi Prathamesh, > -----Original Message----- > From: Prathamesh Kulkarni <prathamesh.kulka...@linaro.org> > Sent: 05 January 2021 11:42 > To: Kyrylo Tkachov <kyrylo.tkac...@arm.com> > Cc: gcc Patches <gcc-patches@gcc.gnu.org> > Subject: Re: [PR66791][ARM] Replace __builtin_vext* with __buitlin_shuffle > in vext intrinsics > > On Mon, 4 Jan 2021 at 16:01, Kyrylo Tkachov <kyrylo.tkac...@arm.com> > wrote: > > > > Hi Prathamesh > > > > > -----Original Message----- > > > From: Prathamesh Kulkarni <prathamesh.kulka...@linaro.org> > > > Sent: 04 January 2021 10:27 > > > To: gcc Patches <gcc-patches@gcc.gnu.org>; Kyrylo Tkachov > > > <kyrylo.tkac...@arm.com> > > > Subject: [PR66791][ARM] Replace __builtin_vext* with __buitlin_shuffle > in > > > vext intrinsics > > > > > > Hi Kyrill, > > > The attached patch replaces __builtin_vextv8qi with __builtin_shuffle > > > for vext_s8. > > > Just wanted to confirm if this is in the correct direction ? > > > If yes, I will send a follow up patch that converts for all vext > > > intrinsics. > > > > Yeah, that does look correct (aarch64 does it that way). > > As before, please make sure to delete any now-unused builtins as well. > Thanks, does the attached patch look OK ?
Ok if testing and bootstrap shows no problems. Thanks, Kyrill > Testing in progress. > > Thanks, > Prathamesh > > > > Thanks, > > Kyrill > > > > > > > > Thanks, > > > Prathamesh
