<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">The llvm-test-suite/SingleSource and llvm-test-suite/MultipleSource does have two failures for AArch64, but they both have nothing to do with Neon.</div>
<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">Some tests like the ones under folder <span style="color:rgb(0,0,0);font-family:Verdana,Helvetica,'sans serif';font-size:11.199999809265137px">MultiSource/Benchmarks/TSVC take extremely long time in foundation model, so they would all fail on timeout, but manually run shows part of them can pass, although I couldn't get full result for them in short time.</span></div>
<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small"><span style="color:rgb(0,0,0);font-family:Verdana,Helvetica,'sans serif';font-size:11.199999809265137px"><br></span></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">
<span style="color:rgb(0,0,0);font-family:Verdana,Helvetica,'sans serif';font-size:11.199999809265137px">In short, I don't think we would have any risk on enabling Neon by default for AArch64.</span></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">
<br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">Thanks,</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">-Jiangning</div></div>
<div class="gmail_extra"><br><br><div class="gmail_quote">2014-01-27 16:51 GMT+08:00 Kristof Beyls <span dir="ltr"><<a href="mailto:kristof.beyls@arm.com" target="_blank">kristof.beyls@arm.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I think the only thing that should still be tested before enabling<br>
AArch64 Neon by default is to check that the LNT test suite doesn't<br>
regress by it, as also suggested by Tim.<br>
<br>
The emperor test suite generates random programs to stress test the<br>
Neon intrinsics implementation. That far majority of all randomly<br>
generated programs now run and pass without problem. I don't think<br>
it's worthwhile to wait until all the corner cases discovered by<br>
the emperor tests are fixed to enable Neon by default.<br>
<br>
In summary, I think the only thing left is for someone to check and<br>
confirm that the test-suite doesn't regress when enabling Neon by<br>
default.<br>
<br>
Thanks,<br>
<br>
Kristof<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
> -----Original Message-----<br>
> From: Ana Pazos [mailto:<a href="mailto:apazos@codeaurora.org">apazos@codeaurora.org</a>]<br>
> Sent: 24 January 2014 19:21<br>
> To: 'Tim Northover'<br>
> Cc: 'llvm-commits'; 'A. Skrobov'; Kristof Beyls<br>
> Subject: RE: [PATCH] [AArch64] Remove NEON from "generic" CPU target<br>
><br>
> Hi tim and Artyom,<br>
><br>
> FYI we have been running several benchmarks (e.g., EEMBC, Spec2000,<br>
> Spec2006, etc.) with slp-vectorizer and neon features enabled.<br>
><br>
> It is looking  good.<br>
><br>
> We had to fix a couple of issues, we are upstreaming the fixes, some of<br>
them<br>
> overlap with the fixes ARM Ltd. has upstreamed as a result of running<br>
> Emperor test.<br>
><br>
> When ARM is done with running Emperor test, I think it will be a good time<br>
> to turn Neon feature on by default.<br>
><br>
> Maybe Kristof can comment when that effort will be complete.<br>
><br>
> Thanks,<br>
> Ana.<br>
><br>
> -----Original Message-----<br>
> From: <a href="mailto:llvm-commits-bounces@cs.uiuc.edu">llvm-commits-bounces@cs.uiuc.edu</a><br>
> [mailto:<a href="mailto:llvm-commits-bounces@cs.uiuc.edu">llvm-commits-bounces@cs.uiuc.edu</a>] On Behalf Of Tim Northover<br>
> Sent: Friday, January 24, 2014 10:40 AM<br>
> To: <a href="mailto:reviews%2BD2110%2Bpublic%2B005b115ff041b29b@llvm-reviews.chandlerc.com">reviews+D2110+public+005b115ff041b29b@llvm-reviews.chandlerc.com</a><br>
> Cc: llvm-commits; A. Skrobov<br>
> Subject: Re: [PATCH] [AArch64] Remove NEON from "generic" CPU target<br>
><br>
> >   Is this the proper time to re-enable NEON in AArch64 by default?<br>
><br>
> Could be. I think the assembly support has been finished for a while.<br>
> If the vectorizer is active (not sure if it needs target stubs<br>
> implementing) it'd be a very good idea to try running the test-suite or<br>
> something though.<br>
><br>
> Cheers.<br>
><br>
> Tim.<br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu">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>
><br>
<br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">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>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><font face="courier new, monospace">Thanks,</font><div><font face="courier new, monospace">-Jiangning</font></div></div>
</div>