Richard Sandiford <richard.sandif...@arm.com> writes:
>> @@ -2034,6 +2034,18 @@ aarch64_expand_fpsr_fpcr_setter (int unspec, 
>> machine_mode mode, tree exp)
>>    emit_insn (gen_aarch64_set (unspec, mode, op));
>>  }
>>  
>> +/* Expand a fpsr or fpcr getter (depending on UNSPEC) using MODE.
>> +   Return the target.  */
>> +static rtx
>> +aarch64_expand_fpsr_fpcr_getter (enum insn_code icode, machine_mode mode,
>> +                             rtx target)
>> +{
>> +  expand_operand op;
>> +  create_output_operand (&op, target, mode);
>> +  expand_insn (icode, 1, &op);
>> +  return target;
>
> This needs to be:
>
>   return op[0].value;

Er, of course I mean op.value.  Muscle memory, sorry. :-)

>
> so that we use whatever target the expand machinery chose.
>
> OK with that change, thanks.
>
> Richard

Reply via email to