<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">Last commitment missed to improve case zext from v1i8 to v1i32 with dup, so committed another patch at r200119 for this case.</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-26 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"><div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">OK. Fixed the following two and committed as r200113.</div>
<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">
1) using "dup b0, v0.b[0]" for v1i8->v1i64</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">2) using "dup h0, v0.h[0]" for v1i16->v1i64</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"><div><div class="h5"><br><br><div class="gmail_quote">2014/1/24 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">
<div>> Attached is to fix pattern match failure for zext/sext from v1xx to v1xx.<br>
<br>
</div>+def : Pat<(v1i64 (zext (v1i8 FPR8:$Rn))),<br>
+        (v1i64 (ANDvvv_8B<br>
+          (v8i8 (SUBREG_TO_REG (i64 0), $Rn, sub_8)),<br>
+          (v8i8 (MOVIdi 1))))>;<br>
<br>
I'm not sure how much of an improvement it would be, but I think<br>
"(v1i64 (SUBREG_TO_REG (i64 0), (DUPbv_8 $Rn), sub_8))" (i.e. a single<br>
"mov bD, vN.b[0]" might be more efficient. Less register pressure and<br>
a single instruction.<br>
<br>
Other than that, it looks good.<br>
<br>
Cheers.<br>
<span><font color="#888888"><br>
Tim.<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br><div dir="ltr"><font face="courier new, monospace">Thanks,</font><div><font face="courier new, monospace">-Jiangning</font></div>
</div>
</font></span></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>