On Tue, Jan 16, 2018 at 04:32:36PM +0000, Wilco Dijkstra wrote: > v2: Rebased after the big SVE commits > > Remove the remaining uses of '*' from aarch64.md. > Using '*' in alternatives is typically incorrect as it tells the register > allocator to ignore those alternatives. Also add a missing '?' so we > prefer a floating point register for same-size int<->fp conversions. > > Passes regress & bootstrap, OK for commit?
Please queue for GCC 9. OK when trunk is back open for new code. Thanks, James > ChangeLog: > 2018-01-16 Wilco Dijkstra <wdijk...@arm.com> > > * config/aarch64/aarch64.md (mov<mode>): Remove '*' in alternatives. > (movsi_aarch64): Likewise. > (load_pairsi): Likewise. > (load_pairdi): Likewise. > (store_pairsi): Likewise. > (store_pairdi): Likewise. > (load_pairsf): Likewise. > (load_pairdf): Likewise. > (store_pairsf): Likewise. > (store_pairdf): Likewise. > (zero_extend): Likewise. > (fcvt_target): Add '?' to prefer w over r. > > gcc/testsuite/ > * gcc.target/aarch64/vfp-1.c: Update test. >