<font size="2"><font face="tahoma,sans-serif">Indeed fixed, thank you.</font></font><div><font class="Apple-style-span" face="tahoma, sans-serif"><br></font></div><div><font size="2"><font face="tahoma,sans-serif"></font></font><font class="Apple-style-span" face="tahoma, sans-serif">ismail<br>

</font><div><br><div class="gmail_quote">On Thu, Dec 2, 2010 at 7:34 PM, Bob Wilson <span dir="ltr"><<a href="mailto:bob.wilson@apple.com">bob.wilson@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div style="word-wrap:break-word">I don't have a linux machine but I think that svn 120716 should fix it.<div><div></div><div class="h5"><div><br><div><div>On Dec 2, 2010, at 12:48 AM, İsmail Dönmez wrote:</div><br><blockquote type="cite">

<font size="2"><font face="tahoma,sans-serif">Hi Bob;</font></font><div><font size="2"><font face="tahoma,sans-serif"><br></font></font></div><div><font size="2"><font face="tahoma,sans-serif">These tests are failing for me on Linux x86-64, looks like it fails to find arm_neon.h.</font></font></div>



<div><font size="2"><font face="tahoma,sans-serif"><br></font></font></div><div><font size="2"><font face="tahoma,sans-serif">Regards,</font></font></div><div><font size="2"><font face="tahoma,sans-serif">ismail<br></font></font><br>



<div class="gmail_quote">On Thu, Dec 2, 2010 at 9:13 AM, Bob Wilson <span dir="ltr"><<a href="mailto:bob.wilson@apple.com" target="_blank">bob.wilson@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



Author: bwilson<br>
Date: Thu Dec  2 01:13:31 2010<br>
New Revision: 120694<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=120694&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=120694&view=rev</a><br>
Log:<br>
Add a test for calling a Neon intrinsic macro with the wrong vector type.<br>
This does not work so well with the -fno-lax-vector-conversions option for<br>
testing the arm_neon.h header but that is a really useful test, so I split<br>
this out to a separate Sema test to check for the warning.<br>
<br>
Added:<br>
    cfe/trunk/test/Sema/arm-neon-types.c<br>
      - copied, changed from r120692, cfe/trunk/test/Headers/arm-neon-header.c<br>
Modified:<br>
    cfe/trunk/test/Headers/arm-neon-header.c<br>
<br>
Modified: cfe/trunk/test/Headers/arm-neon-header.c<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Headers/arm-neon-header.c?rev=120694&r1=120693&r2=120694&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Headers/arm-neon-header.c?rev=120694&r1=120693&r2=120694&view=diff</a><br>




==============================================================================<br>
--- cfe/trunk/test/Headers/arm-neon-header.c (original)<br>
+++ cfe/trunk/test/Headers/arm-neon-header.c Thu Dec  2 01:13:31 2010<br>
@@ -3,8 +3,3 @@<br>
 // RUN: %clang_cc1 -x c++ -triple thumbv7-apple-darwin10 -target-cpu cortex-a8 -fsyntax-only -Wvector-conversions -verify %s<br>
<br>
 #include <arm_neon.h><br>
-<br>
-// Radar 8228022: Should not report incompatible vector types.<br>
-int32x2_t test(int32x2_t x) {<br>
-  return vshr_n_s32(x, 31);<br>
-}<br>
<br>
Copied: cfe/trunk/test/Sema/arm-neon-types.c (from r120692, cfe/trunk/test/Headers/arm-neon-header.c)<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Sema/arm-neon-types.c?p2=cfe/trunk/test/Sema/arm-neon-types.c&p1=cfe/trunk/test/Headers/arm-neon-header.c&r1=120692&r2=120694&rev=120694&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Sema/arm-neon-types.c?p2=cfe/trunk/test/Sema/arm-neon-types.c&p1=cfe/trunk/test/Headers/arm-neon-header.c&r1=120692&r2=120694&rev=120694&view=diff</a><br>




==============================================================================<br>
--- cfe/trunk/test/Headers/arm-neon-header.c (original)<br>
+++ cfe/trunk/test/Sema/arm-neon-types.c Thu Dec  2 01:13:31 2010<br>
@@ -1,6 +1,4 @@<br>
 // RUN: %clang_cc1 -triple thumbv7-apple-darwin10 -target-cpu cortex-a8 -fsyntax-only -Wvector-conversions -verify %s<br>
-// RUN: %clang_cc1 -triple thumbv7-apple-darwin10 -target-cpu cortex-a8 -fsyntax-only -fno-lax-vector-conversions -verify %s<br>
-// RUN: %clang_cc1 -x c++ -triple thumbv7-apple-darwin10 -target-cpu cortex-a8 -fsyntax-only -Wvector-conversions -verify %s<br>
<br>
 #include <arm_neon.h><br>
<br>
@@ -8,3 +6,8 @@<br>
 int32x2_t test(int32x2_t x) {<br>
   return vshr_n_s32(x, 31);<br>
 }<br>
+<br>
+// ...but should warn when the types really do not match.<br>
+float32x2_t test2(uint32x2_t x) {<br>
+  return vcvt_n_f32_s32(x, 0); // expected-warning {{incompatible vector types}}<br>
+}<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br></div>
</blockquote></div><br></div></div></div></div></blockquote></div><br></div></div>