<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">committed as r199296.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014/1/13 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;display:inline">
Hi </div>Ana,<br><br>I have two reasons for handling this situation in "combine"<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small;display:inline">
 stage.</div><br>1) If we customize lowering sext_inreg, we would probably missing the case of explicitly using shl/ashr in LLVM IR. I added three more cases with suffix "_2" to cover this kind of test. If we want to cover all in legalization, we would have to write more codes, I think.<br>

2) LLVM documentation says "Subsequent runs of the pass clean up the messy code generated by the Legalize passes, which allows Legalize to be very simple (it can focus on making code legal instead of focusing on generating good and legal code)."<br>

<br><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">Kevin already upstreamed the improvement of generating permutation instructions, so now I'm uploading new patches by</div>
<div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">
1) Checking uzip1 in the test</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small">2) Adding three more cases for explicity shl/ashr pair.</div><br><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><br><div class="gmail_extra">
</div></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>