On Tue, May 15, 2012 at 1:47 PM, Jakub Jelinek <ja...@redhat.com> wrote:
> This is a similar problem as PR52086, but back then when looking > at whether other peepholes might need similar treatment I didn't think > about the possibility that addqi_1 pattern would allow non-q constraint > register. Apparently it does in some of the alternatives, the insn is then > emitted widened as addl etc., but for addqi_2 it isn't possible (it adds to > memory). > > Fixed thusly, bootstrapped/regtested on i686-linux, ok for trunk/4.7? > > 2012-05-15 Jakub Jelinek <ja...@redhat.com> > > PR target/53358 > * config/i386/i386.md (*addqi_2 peephole with QImode addition): Check > that operands[2] is either immediate, or q_regs_operand. > > * gcc.dg/pr53358.c: New test. OK. Thanks, Uros.