Tamar Christina <tamar.christ...@arm.com> writes: > Hi Richard, > > The 02/11/2019 18:22, Richard Sandiford wrote: >> pollute the target-independent namespace). Then we can use something like: >> >> expand_operand ops[3]; >> create_output_operand (&ops[0], dest, mode); >> create_input_operand (&ops[1], pred, GET_MODE (pred)); >> create_input_operand (&ops[2], src, mode); >> expand_insn (code_for_aarch64_pred_mov (mode), 3, ops); >> >> (completely untested). >> > > Oh, I didn't know about these, thanks! > > Bootstrapped on aarch64-none-linux-gnu and no issues. > Regtested on aarch64-none-elf with SVE on and no issues. > > Ok for trunk?
OK, thanks. Richard > > Thanks, > Tamar > > gcc/ChangeLog: > > 2019-02-13 Tamar Christina <tamar.christ...@arm.com> > > PR target/88847 > * config/aarch64/aarch64-sve.md (*pred_mov<mode>, pred_mov<mode>): > Expose as @aarch64_pred_mov. > * config/aarch64/aarch64.c (aarch64_classify_address): > Use expand_insn which legitimizes operands. > > gcc/testsuite/ChangeLog: > > 2019-02-13 Tamar Christina <tamar.christ...@arm.com> > > PR target/88847 > * gcc.target/aarch64/sve/pr88847.c: New test. > >> Thanks, >> Richard