On 12 November 2014 16:46, Ramana Radhakrishnan <ramana.radhakrish...@arm.com> wrote:
> v2 , based on Richard's suggestion as well as fixing a bug that I hit in > some more testing at O1. aarch64_internal_mov_immediate should not generate > a temporary for subtarget when not actually "generating" code. > > Tested again on aarch64-none-elf and with a bootstrap / reg test. Ok ? > > > <DATE> Ramana Radhakrishnan <ramana.radhakrish...@arm.com> > > PR target/63724 > * config/aarch64/aarch64.c (aarch64_expand_mov_immediate): Split out > numerical immediate handling to... > (aarch64_internal_mov_immediate): ...this. New. > (aarch64_rtx_costs): Use aarch64_internal_mov_immediate. > (aarch64_mov_operand_p): Relax predicate. > * config/aarch64/aarch64.md (mov<mode>:GPI): Do not expand > CONST_INTs. > (*movsi_aarch64): Turn into define_insn_and_split and new > alternative > for 'n'. > (*movdi_aarch64): Likewise. OK. We should be able to remove aarch64_build_constant now. /Marcus
