On Fri, Oct 3, 2014 at 12:26 PM, Kirill Yukhin <kirill.yuk...@gmail.com> wrote: > Hello Uroš, > On 29 Sep 09:54, Uros Bizjak wrote: >> > +(define_expand "vashrv2di3<mask_name>" >> > + [(set (match_operand:V2DI 0 "register_operand") >> > + (ashiftrt:V2DI >> > + (match_operand:V2DI 1 "register_operand") >> > + (match_operand:V2DI 2 "nonimmediate_operand")))] >> > + "TARGET_XOP || TARGET_AVX512VL" >> > +{ >> > + if (!TARGET_XOP) >> >> This condition is wrong. Please re-test the patch. > Great catch! Didn't tested whole i386.exp, so XOP tests didn't run. > Fixed. Patch in the bottom. XOP tests are now pass. > Is it ok for trunk now?
OK. Thanks, Uros.