Richard Earnshaw (lists) wrote: > (define_insn "*movdi_vfp" > - [(set (match_operand:DI 0 "nonimmediate_di_operand" > "=r,r,r,r,q,q,m,w,r,w,w, Uv") > + [(set (match_operand:DI 0 "nonimmediate_di_operand" > "=r,r,r,r,q,q,m,w,!r,w,w, Uv")
> Why have you introduced a no-reloads block on the 9th alternative for > all variants? That is the default behaviour when you don't explicitly set a cpu, so I kept that. See https://patches.linaro.org/patch/541/ for the original reason for adding it - duplicating this pattern was a mistake since '!' wouldn't pessimize other cores as int<->fp moves typically have a non-trivial cost. However given Cortex-A8 is ancient now we could just remove the '!'. Wilco
