<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="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>