On Fri, Sep 5, 2014 at 1:48 PM, Segher Boessenkool <seg...@kernel.crashing.org> wrote: > I made a thinko in r214080. That patch makes the "andi." pattern not match > for insns that can match "rlwinm", and also makes "rlwinm." not match insns > that can match "andi." (and similar for "andis." and "rldicl" etc.) This > ensures we always choose the cheapest instruction. > > However, the splitter for "andi." does not take this into account, and > generates a clobber always, leading to unrecognisable instructions. This > patch fixes it. > > Okay if it survives bootstrap+regcheck? > > > Segher > > > 2014-09-05 Segher Boessenkool <seg...@kernel.crashing.org> > > PR target/63187 > * config/rs6000/rs6000.md (*and<mode>3_imm_dot, *and<mode>3_imm_dot2): > Do not allow any_mask_operand for operands[2]. > (*and<mode>3_imm_mask_dot, *and<mode>3_imm_mask_dot2): New.
Okay. Thanks, David
