Hi Christophe, > -----Original Message----- > From: Christophe Lyon <[email protected]> > Sent: Thursday, June 28, 2018 14:41 > To: James Greenhalgh <[email protected]> > Cc: Tamar Christina <[email protected]>; Kyrill Tkachov > <[email protected]>; gcc Patches <[email protected]>; > nd <[email protected]>; Richard Earnshaw <[email protected]>; > Marcus Shawcroft <[email protected]> > Subject: Re: [PATCH][GCC][AArch64] Add SIMD to REG pattern for movhf > without armv8.2-a support (PR85769) > > On Tue, 26 Jun 2018 at 23:53, James Greenhalgh > <[email protected]> wrote: > > > > On Wed, Jun 20, 2018 at 05:15:37AM -0500, Tamar Christina wrote: > > > Hi Kyrill, > > > > > > Many thanks for the review! > > > > > > The 06/19/2018 16:47, Kyrill Tkachov wrote: > > > > Hi Tamar, > > > > > > > > On 19/06/18 15:07, Tamar Christina wrote: > > > > > Hi All, > > > > > > > > > > This fixes a regression where we don't have an instruction for > > > > > pre Armv8.2-a to do a move of an fp16 value from a GP reg to a SIMD > reg. > > > > > > > > > > This patch adds that pattern to movhf_aarch64 using a dup and > > > > > only selectes it using a very low priority. > > > > > > > > > > This fixes an ICE at -O0. > > > > > > > > > > Regtested on aarch64-none-elf and no issues. > > > > > Bootstrapped on aarch64-none-linux-gnu and no issues. > > > > > > > > > > Ok for master? > > > > OK, > > > > Hi, > > I've noticed that the new test fails: > FAIL: gcc.target/aarch64/f16_mov_immediate_3.c scan-assembler-times > dup\\tv[0-9]+.4h, w[0-9]+ 1 > > It might be a matter of old binutils not supporting v8.2, and maybe your test > should use an effective target to check that?
No, it was that Wilco had an unrelated patch that changed/fixed the costing in the register allocator. Due to it the target started generating load/stores again and so the constants I was testing being so simple it didn't generate a dup anymore. When I applied the patch on trunk it changed the behaviour. Wilco has fixed the test already by making the constant more complex. Regards, Tamar > > Thanks, > > Christophe > > > Thanks, > > James > > > > > > > > > > > > gcc/ > > > > > 2018-06-19 Tamar Christina <[email protected]> > > > > > > > > > > PR target/85769 > > > > > * config/aarch64/aarch64.md (*movhf_aarch64): Add dup v0.4h > pattern. > > > > > > > > > > gcc/testsuite/ > > > > > 2018-06-19 Tamar Christina <[email protected]> > > > > > > > > > > PR target/85769 > > > > > * gcc.target/aarch64/f16_mov_immediate_3.c: New. > > > > > Thanks, > > > > > Tamar > > > > > > > > > > -- > > >
