Hi Carl, On Mon, Dec 18, 2017 at 04:10:06PM -0800, Carl Love wrote: > --- a/gcc/config/rs6000/altivec.md > +++ b/gcc/config/rs6000/altivec.md > @@ -1,3 +1,4 @@ > + > ;; AltiVec patterns. > ;; Copyright (C) 2002-2017 Free Software Foundation, Inc. > ;; Contributed by Aldy Hernandez (a...@quesejoda.com)
Please lose this change. > +;; Power8 vector merge two V2DF/V2DI even words to V2DF > +(define_expand "p8_vmrgew_<mode>" > + [(use (match_operand:VSX_D 0 "vsx_register_operand" "")) > + (use (match_operand:VSX_D 1 "vsx_register_operand" "")) > + (use (match_operand:VSX_D 2 "vsx_register_operand" ""))] Drop the empty default field please (here and elsewhere, certainly in all expanders). It looks good to me otherwise. Okay for trunk. Thanks! Segher