<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">Hi Zhaoshi,</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">For both little-endian and big-endian, we need to generate instructions as if ldr/str. For little-endian, ldr/str is the same as ld1/st1, while for big-endian they have different behaviours. So if we generate ld1/st1 for big-endian, we should have to generate "rev" following them to keep the reversed in-register layout.</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">Your code shows you are saying Linux should be always configured as non-strict mode for hardware, so for big-endian the choice between "ldr/str" and "ld1/st1 + rev" will depend on cost estimate. Therefore, for your test case unaligned-vector-ld1-st1.ll,</div>
<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small"><br></div><div class="gmail_default"><font face="arial, helvetica, sans-serif">+; RUN: llc < %s -mtriple=aarch64_be-none-linux-gnu -mattr=+neon -o - | FileCheck %s</font><br>
<br></div><div class="gmail_default"><span style="font-family:arial,helvetica,sans-serif">It would be better if you can add -</span><font face="arial, helvetica, sans-serif">aarch64-strict-align in command line, and the ld1/st1 CHECK should be guaranteed for both little-endian and big-endian. Also, I don't see you have tests covering those three newly added opts you added.</font></div>
<div class="gmail_default"><br></div><div class="gmail_default"><font face="arial, helvetica, sans-serif">Thanks,</font></div><div class="gmail_default"><font face="arial, helvetica, sans-serif">-Jiangning</font></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">
<br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-04-08 9:24 GMT+08:00  <span dir="ltr"><<a href="mailto:zhaoshiz@codeaurora.org" target="_blank">zhaoshiz@codeaurora.org</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
This patch should enable unaligned accesses of vector types on AArch64.<br>
Please help review.<br>
<br>
Thanks,<br>
Zhaoshi<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><div dir="ltr"><font face="courier new, monospace">Thanks,</font><div><font face="courier new, monospace">-Jiangning</font></div></div>
</div>