Re: [PATCH 6/6] rs6000: Merge the var_shift yes/no alternatives

2014-06-22 Thread David Edelsohn
On Sat, Jun 21, 2014 at 10:47 PM, Segher Boessenkool wrote: > All instructions that are "var_shift" for some alternative have the shift > amount as operands[2]. > > This patch introduces an attribute "maybe_var_shift". If that is set to > "yes", the default value of "var_shift" is set based on th

Re: [PATCH 6/6] rs6000: Merge the var_shift yes/no alternatives

2014-06-22 Thread Segher Boessenkool
> (define_insn "rotl3" > - [(set (match_operand:GPR 0 "gpc_reg_operand" "=r,r") > - (rotate:GPR (match_operand:GPR 1 "gpc_reg_operand" "r,r") > - (match_operand:GPR 2 "reg_or_cint_operand" "r,n")))] > + [(set (match_operand:GPR 0 "gpc_reg_operand" "=r") > + (rotate:GPR (m

[PATCH 6/6] rs6000: Merge the var_shift yes/no alternatives

2014-06-21 Thread Segher Boessenkool
All instructions that are "var_shift" for some alternative have the shift amount as operands[2]. This patch introduces an attribute "maybe_var_shift". If that is set to "yes", the default value of "var_shift" is set based on the operands[2] value. With that, we can merge the var_shift yes/no cas