<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">Hi Tim,</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small"> </div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">
Are you talking about the intrinsics without digit suffix like <span><span style="color:rgb(0,0,0);text-transform:none;text-indent:0px;letter-spacing:normal;word-spacing:0px;float:none;display:inline!important;white-space:normal">vuzp_s8</span>? They looks like legacy for me. If yes, I think this is already covered. I have test like the followings,</span></div>
<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small"><span></span> </div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small"><span>int8x8x2_t test_vuzp_s8(int8x8_t a, int8x8_t b) {<br>
  // CHECK: test_vuzp_s8<br>  return vuzp_s8(a, b);<br>  // CHECK: uzp1 {{v[0-9]+}}.8b, {{v[0-9]+}}.8b<br>  // CHECK: uzp2 {{v[0-9]+}}.8b, {{v[0-9]+}}.8b<br>}</span></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">
<span></span> </div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small"><span>Thanks,</span></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">
<span>-Jiangning</span></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small"><span></span> </div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/11/6 Tim Northover <span dir="ltr"><<a href="mailto:t.p.northover@gmail.com" target="_blank">t.p.northover@gmail.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  Hi Jiangning,<br>
<br>
  I've got one question (well, stated as a comment) about this Clang stuff.<br>
<br>
  Cheers.<br>
<br>
  Tim.<br>
<br>
<br>
================<br>
Comment at: lib/CodeGen/CGBuiltin.cpp:2374-2375<br>
@@ -2373,1 +2373,4 @@<br>
   // in aarch64-neon-intrinsics.c so far.<br>
+  case AArch64::BI__builtin_neon_vuzp_v:<br>
+    return EmitARMBuiltinExpr(ARM::BI__builtin_neon_vuzp_v, E);<br>
+  case AArch64::BI__builtin_neon_vuzpq_v:<br>
----------------<br>
If I'm reading correctly these cases (the legacy ARM zip/uzp/trn intrinsics) are neither tested in this patch nor implemented on the LLVM side.<br>
<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D2075" target="_blank">http://llvm-reviews.chandlerc.com/D2075</a><br>
</blockquote></div><br><br clear="all"><br>-- <br><div dir="ltr"><font face="courier new, monospace">Thanks,</font><div><font face="courier new, monospace">-Jiangning</font></div></div>
</div>