Richard Sandiford <richard.sandif...@arm.com> writes: > 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
Installed in trunk as f5e73de00e9. Thanks! Andrea