<div dir="ltr"><div class="gmail_default"><font face="arial, helvetica, sans-serif">Tim,</font></div><div class="gmail_default"><font face="arial, helvetica, sans-serif"><br></font></div><div class="gmail_default"><font face="arial, helvetica, sans-serif">You are correct. I made the change and committed as r195408 and </font><span style="font-family:arial,helvetica,sans-serif">r195409.</span><br>
</div><div class="gmail_default"><span style="font-family:arial,helvetica,sans-serif"><br></span></div><div class="gmail_default"><span style="font-family:arial,helvetica,sans-serif">Thanks,</span></div><div class="gmail_default">
<span style="font-family:arial,helvetica,sans-serif">-Jiangning</span></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/11/21 Tim Northover <span dir="ltr"><<a href="mailto:t.p.northover@gmail.com" target="_blank">t.p.northover@gmail.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Jiangning,<br>
<div class="im"><br>
On 18 November 2013 10:48, Jiangning Liu <<a href="mailto:liujiangning1@gmail.com">liujiangning1@gmail.com</a>> wrote:<br>
> Attached patch is to implement AArch64 SISD intrinsics for vget_high and<br>
> vget_low.<br>
<br>
</div>+def Neon_High2D  : PatFrag<(ops node:$in),<br>
+                           (extract_subvector (v2i64 node:$in), (iPTR 1))>;<br>
+def Neon_High2d  : PatFrag<(ops node:$in),<br>
+                           (extract_subvector (v2f64 node:$in), (iPTR 1))>;<br>
<br>
These two should not be named so similarly. There's also (existing,<br>
but shouldn't) capitalisation confusion in that area: Neon_High vs<br>
Neon_low.<br>
<br>
+  def : Pat<(ResTy (GetLow VPR128:$Rn)),<br>
+            (ResTy (DUPdv_D (OpTy VPR128:$Rn), 0))>;<br>
<br>
This should be a simple EXTRACT_SUBREG shouldn't it? That way the<br>
register allocator can almost certainly remove it completely. In<br>
particular I'd expect all of your tests to be empty functions: "ret"<br>
only.<br>
<br>
Cheers.<br>
<span class="HOEnZb"><font color="#888888"><br>
Tim.<br>
</font></span></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>