2015-02-17 14:12 GMT+03:00 Georg-Johann Lay <a...@gjlay.de>:
> Byte-wise pushing virtual regs like arg pointer migth result in patterns
> like
>
>  (set (mem:QI (post_dec:HI (reg:HI 32 SP)))
>       (subreg:QI (plus:HI (reg:HI 28)
>                           (const_int 17)) 0))
>
> after elimination.
>
> Attached patch uses new pushhi1_insn to push virtuals in HImode so that
> expressions like in subreg_reg from above can be reloaded.
>
> Ok to commit ?
>
> Johann
>
>         PR target/64452
>
>         * config/avr/avr.md (pushhi_insn): New insn.
>         (push<mode>1): Push virtual regs in one chunk using pushhi1_insn.

Approved.
(But I'm worry about this because it's reload related problem and it
can have a side effect)

Denis.

Reply via email to