<div dir="ltr"><div dir="ltr" style="font-size:12.8px">Hi Zvi,<div><br></div><div>Review patch was generated against my sandbox which was last updated on 17/07/2017. Will rebase with trunk top. </div><div><br></div><div>This diff makes vector argument splitting efficient as it uses only minimal required subvector needed to create a vector_shuffle.</div><div><br></div><div>Regards,</div><div>Jatin</div><div><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 25, 2017 at 10:27 AM, Soft Dev <span dir="ltr"><<a href="mailto:soft.devl81@gmail.com" target="_blank">soft.devl81@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Zvi,<div><br></div><div>Review patch was generated against my sandbox which was last updated on 17/07/2017. Will rebase with trunk top. </div><div><br></div><div>This diff makes vector argument splitting efficient as it uses only minimal required subvector needed to create a vector_shuffle.</div><div><br></div><div>Regards.   </div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 25, 2017 at 1:41 AM, Zvi Rackover via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">zvi added a comment.<br>
<br>
See <a href="https://reviews.llvm.org/D35700" rel="noreferrer" target="_blank">https://reviews.llvm.org/D3570<wbr>0</a> where pr33784 is handled a bit differently.<br>
<span><br>
<br>
<br>
================<br>
Comment at: test/CodeGen/X86/pr33784-vecto<wbr>r-shuffle.ll:1<br>
+; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.p<wbr>y<br>
+<br>
----------------<br>
</span><span>RKSimon wrote:<br>
> zvi wrote:<br>
> > Can you please commit the test and re-apply the patch so that we can see the change in the generated code?<br>
> Please can you add this test to shuffle-vs-trunc-512.ll instead of creating a new test file?<br>
</span>This case already exists in test/CodeGen/X86/shuffle-strid<wbr>ed-with-offset-512.ll?<br>
Can you please verify you are rebased on top-of-trunk?<br>
<br>
<br>
<a href="https://reviews.llvm.org/D35788" rel="noreferrer" target="_blank">https://reviews.llvm.org/D3578<wbr>8</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>
</blockquote></div><br></div>