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

Reply via email to