On Tue, Sep 20, 2016 at 07:38:43PM -0400, Michael Meissner wrote: > Since this is in ISA 3.0 code only, I did not do the full bootstrap and make > check sequence. I can do this if you prefer.
Yes, please do, to make sure you didn't accidentally break something else. > Is the patch ok to put into the trunk? > +(define_insn_and_split "*mov<SFDF:mode><SFDF2:mode>cc_invert_p9" > + [(set (match_operand:SFDF 0 "vsx_register_operand" "=&<SFDF:Fv>,<SFDF:Fv>") > + (if_then_else:SFDF > + (match_operator:CCFP 1 "invert_fpmask_comparison_operator" > + [(match_operand:SFDF2 2 "vsx_register_operand" > "<SFDF2:Fv>,<SFDF2:Fv>") > + (match_operand:SFDF2 3 "vsx_register_operand" > "<SFDF2:Fv>,<SFDF2:Fv>")]) > + (match_operand:SFDF 4 "vsx_register_operand" "<SFDF:Fv>,<SFDF:Fv>") > + (match_operand:SFDF 5 "vsx_register_operand" "<SFDF:Fv>,<SFDF:Fv>"))) > + (clobber (match_scratch:V2DI 6 "=0,&wa"))] > + "TARGET_P9_MINMAX" > + "#" > + "" "&& 1" here? Okay with that change. Thanks, Segher
