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.

Reply via email to