On Thu, 15 Jul 2021 at 14:47, Christophe Lyon <christophe.lyon....@gmail.com> wrote: > > Hi Prathamesh, > > On Mon, Jul 5, 2021 at 11:25 AM Kyrylo Tkachov via Gcc-patches > <gcc-patches@gcc.gnu.org> wrote: >> >> >> >> > -----Original Message----- >> > From: Prathamesh Kulkarni <prathamesh.kulka...@linaro.org> >> > Sent: 05 July 2021 10:18 >> > To: gcc Patches <gcc-patches@gcc.gnu.org>; Kyrylo Tkachov >> > <kyrylo.tkac...@arm.com> >> > Subject: [ARM] PR66791: Replace builtins for fp and unsigned vmul_n >> > intrinsics >> > >> > Hi Kyrill, >> > I assume this patch is OK to commit after bootstrap+testing ? >> >> Yes. >> Thanks, >> Kyrill >> > > > The updated testcase fails on some configs: > gcc.target/arm/armv8_2-fp16-neon-2.c: vdup\\.16\\tq[0-9]+, r[0-9]+ found 2 > times > FAIL: gcc.target/arm/armv8_2-fp16-neon-2.c scan-assembler-times > vdup\\.16\\tq[0-9]+, r[0-9]+ 3 > > For instance on arm-none-eabi with default configuration flags (mode/cpu/fpu) > and default runtestflags. > The same toolchain config also fails on this test when overriding > runtestflags with: > -mthumb/-mfloat-abi=soft/-march=armv6s-m > -mthumb/-mfloat-abi=soft/-march=armv7-m > -mthumb/-mfloat-abi=soft/-march=armv8.1-m.main > > Can you fix this please? Hi Christophe, Sorry for the breakage, I will take a look.
Thanks, Prathamesh > > Thanks, > > Christophe > >> > >> > Thanks, >> > Prathamesh