Hi Pirama,<br><br>This looks OK to me, but could you please add a test for v8f16 and v16f16 to check oversize vectors are handled sanely? I'm trying to keep the backend oversize-safe. <br><br>Cheers,<br><br>James<br><div class="gmail_quote">On Tue, 21 Apr 2015 at 20:44, Pirama Arumuga Nainar <<a href="mailto:pirama@google.com">pirama@google.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">IIUC, this patch shouldn't affect v4i32 to v4f32 or v4f64 operations, but couldn't find any tests for those operations. I confirmed that locally, and can add tests with this patch if requested.<br>
<br>
In a test of v4f16 operations (similar to the comprehensive test in <a href="http://reviews.llvm.org/D8648" target="_blank">http://reviews.llvm.org/D8648</a>), only sitofp and uitofp cause problems, and will be fixed with this patch.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D9166" target="_blank">http://reviews.llvm.org/D9166</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>