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