<div dir="ltr"><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">2013/11/19 Jiangning Liu <span dir="ltr"><<a href="mailto:liujiangning1@gmail.com" target="_blank">liujiangning1@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 Hao,<br>
<br>
  Overall LGTM. For those two comments I gave I think you can do a refactoring work later on after committing this one.<br>
<br>
  Since LLVM 3.4 branch is going to be created in an hour, and this patch is the only big feature we are missing for AArch64 Neon, I think you should go ahead to get it committed ASAP.<br>
<br>
  Thanks,<br>
  -Jiangning<br>
<br>
<br>
================<br>
Comment at: lib/Target/AArch64/AArch64ISelDAGToDAG.cpp:1271<br>
@@ -1049,1 +1270,3 @@<br>
   }<br>
+  case AArch64ISD::NEON_LD2DUP: {<br>
+    static const uint16_t Opcodes[] = { AArch64::LD2R_8B,  AArch64::LD2R_4H,<br>
----------------<br>
This piece of code logic can be combined together with line 815, because they are quite related.  It would be better to centralize the relevant logic.<br>
<br>
================<br>
Comment at: lib/Target/AArch64/AArch64ISelDAGToDAG.cpp:1533<br>
@@ -1212,1 +1532,3 @@<br>
     }<br>
+    case Intrinsic::arm_neon_vld2lane: {<br>
+      static const uint16_t Opcodes[] = { AArch64::LD2LN_B, AArch64::LD2LN_H,<br>
----------------<br>
The same comments as line 1271.<br>
<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D2211" target="_blank">http://llvm-reviews.chandlerc.com/D2211</a><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>