<div dir="ltr">LGTM. Thanks!</div><div class="gmail_extra"><br><div class="gmail_quote">2014-11-14 8:31 GMT+03:00 Cameron McInally <span dir="ltr"><<a href="mailto:cameron.mcinally@nyu.edu" target="_blank">cameron.mcinally@nyu.edu</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Tests updated. Also updated the CHECK statements to resemble the<br>
valign patterns.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
On Thu, Nov 13, 2014 at 11:21 PM, Cameron McInally<br>
<<a href="mailto:cameron.mcinally@nyu.edu">cameron.mcinally@nyu.edu</a>> wrote:<br>
> Ugh, sorry. I was 1/2 asleep. I caught this just after sending. Here<br>
> is the updated patch...<br>
><br>
> I'll also fix up the tests.<br>
><br>
> -Cam<br>
><br>
> On Thu, Nov 13, 2014 at 11:17 PM, Robert Khasanov<br>
> <<a href="mailto:rob.khasanov@gmail.com">rob.khasanov@gmail.com</a>> wrote:<br>
>> In addition to previous:<br>
>><br>
>> 3) In zero-masking tests I see that masks are not passed through args.<br>
>> Please rewrite them.<br>
>> +define <16 x i32> @test_x86_avx512_maskz_psrai_d(<16 x i32> %a0) {<br>
>> +  ; CHECK-LABEL: test_x86_avx512_maskz_psrai_d<br>
>> +  ; CHECK: vpsrad  $7, %zmm{{[0-9]+}}, %zmm{{[0-9]+}} {%k{{[0-9]+}}} {z}<br>
>> +  %res = call <16 x i32> @llvm.x86.avx512.mask.psrai.d(<16 x i32> %a0, i32<br>
>> 7, <16 x i32> zeroinitializer, i16 2)<br>
>> +  ret <16 x i32> %res<br>
>> +}<br>
>><br>
>> For merge-masking tests please pass also pass-through value through args.<br>
>><br>
>> 2014-11-14 7:07 GMT+03:00 Robert Khasanov <<a href="mailto:rob.khasanov@gmail.com">rob.khasanov@gmail.com</a>>:<br>
>>><br>
>>> Hi Cameron,<br>
>>><br>
>>> 1) Please remove arguments from avx512_shift_rmi that would be not needed:<br>
>>> RC. You can get vt, x86memop and mem_frag from X86VectorVTInfo, they could<br>
>>> be also removed from args.<br>
>>> 2) I see you only added zero-masking tests. Please include also<br>
>>> merge-masking tests.<br>
>>><br>
>>> 2014-11-14 6:58 GMT+03:00 Cameron McInally <<a href="mailto:cameron.mcinally@nyu.edu">cameron.mcinally@nyu.edu</a>>:<br>
>>>><br>
>>>> Hi guys,<br>
>>>><br>
>>>> Here is a patch to add masked patterns for 512b integer shift by<br>
>>>> immediate.<br>
>>>><br>
>>>> Thanks,<br>
>>>> Cam<br>
>>><br>
>>><br>
>><br>
</div></div></blockquote></div><br></div>