<div dir="ltr">In addition to previous: <div><br></div><div>3) In zero-masking tests I see that masks are not passed through args. Please rewrite them. </div><div><div>+define <16 x i32> @test_x86_avx512_maskz_psrai_d(<16 x i32> %a0) {</div><div>+  ; CHECK-LABEL: test_x86_avx512_maskz_psrai_d</div><div>+  ; CHECK: vpsrad  $7, %zmm{{[0-9]+}}, %zmm{{[0-9]+}} {%k{{[0-9]+}}} {z}</div><div>+  %res = call <16 x i32> @llvm.x86.avx512.mask.psrai.d(<16 x i32> %a0, i32 7, <16 x i32> zeroinitializer,<b> i16 2</b>)</div><div>+  ret <16 x i32> %res</div><div>+}</div></div><div><br></div><div>For merge-masking tests please pass also pass-through value through args. </div></div><div class="gmail_extra"><br><div class="gmail_quote">2014-11-14 7:07 GMT+03:00 Robert Khasanov <span dir="ltr"><<a href="mailto:rob.khasanov@gmail.com" target="_blank">rob.khasanov@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Cameron,<div><br></div><div>1) Please remove arguments from avx512_shift_rmi that would be not needed: RC. You can get vt, x86memop and mem_frag from X86VectorVTInfo, they could be also removed from args.</div><div>2) I see you only added zero-masking tests. Please include also merge-masking tests.</div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">2014-11-14 6:58 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">Hi guys,<br>
<br>
Here is a patch to add masked patterns for 512b integer shift by immediate.<br>
<br>
Thanks,<br>
Cam<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>