<div dir="ltr"><span style="font-family:arial,sans-serif;font-size:14.399999618530273px">Hi Ana,</span><div style="font-family:arial,sans-serif;font-size:14.399999618530273px"><br></div><div style="font-family:arial,sans-serif;font-size:14.399999618530273px">
The patch looks good for me.</div><div style="font-family:arial,sans-serif;font-size:14.399999618530273px"><br></div><div style="font-family:arial,sans-serif;font-size:14.399999618530273px">The MCHammer test result shows encoding/decoding and assemble/disassemble can all pass for the following three instruction classes,<br>
</div><div style="font-family:arial,sans-serif;font-size:14.399999618530273px"><br></div><div style="font-family:arial,sans-serif;font-size:14.399999618530273px"><div>* Vector Arithmetic</div><div>* Vector Immediate</div>
<div>* Vectore Pairwise Arithmetic</div></div><div style="font-family:arial,sans-serif;font-size:14.399999618530273px"><br></div><div style="font-family:arial,sans-serif;font-size:14.399999618530273px">There is a typo for the last one in your commit log. Remove 'e' from Vectore.</div>
<div style="font-family:arial,sans-serif;font-size:14.399999618530273px"><br></div><div style="font-family:arial,sans-serif;font-size:14.399999618530273px">BTW, it would be better to add tag [PATCH] in your subject, because all of patches in this mail list are with this tag.</div>
<div style="font-family:arial,sans-serif;font-size:14.399999618530273px"><br></div><div style="font-family:arial,sans-serif;font-size:14.399999618530273px">Thanks,</div><div style="font-family:arial,sans-serif;font-size:14.399999618530273px">
-Jiangning</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/7/31 Ana Pazos <span dir="ltr"><<a href="mailto:apazos@codeaurora.org" target="_blank">apazos@codeaurora.org</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hello Tim and LLVM reviewers,<br>
<br>
I have updated the Clang and aarch64 backend patches to support arm v8 Neon<br>
to merge cleanly with the latest Clang/LLVM code.<br>
<br>
You have reviewed these patches before, but they have not been merged yet.<br>
We would like to have them merged now.<br>
<br>
Tim (and anyone else interested), can you do the review of the rebased<br>
patches and merge them?<br>
<br>
Thanks,<br>
Ana.<br>
<br>
<br>
-----Original Message-----<br>
From: Tim Northover [mailto:<a href="mailto:t.p.northover@gmail.com">t.p.northover@gmail.com</a>]<br>
Sent: Thursday, January 31, 2013 10:52 AM<br>
To: Ana Pazos<br>
Cc: <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a>; llvm-commits; <a href="mailto:anapazos@codeaurora.org">anapazos@codeaurora.org</a><br>
Subject: Re: clang and aarch64 backend changes to support Neon<br>
<br>
Hi Ana,<br>
<br>
Thanks for the patches.<br>
<br>
> Here are the initial clang and aarch64 backend changes to support Neon.<br>
<br>
Just a note for others that I believe these are the changes that will bring<br>
the AArch64 backend as committed in line with what has already been reviewed<br>
so far. I don't think they need separate technical review (though obviously<br>
if anyone *does* have comments...).<br>
<br>
> If everyone is ok with these changes, can you Tim please go ahead and<br>
> commit them?<br>
<br>
As soon as I'm allowed, I will.<br>
<br>
Tim.<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>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br>Thanks,<div>-Jiangning</div>
</div>