<div dir="ltr">Hi Tim,<div><br></div><div>Here come new patches for review.</div><div><br></div><div>1) Changed some naming conventions in both clang and llvm patches. For example, change _High to _Hi in clang, and change _8H to _8h8b in .td file of llvm.</div>
<div>2) Change all addw and subw to be not commutable</div><div>3) Modified pattern match for instructions relevant to the followings, </div><div><br></div><div><div style="font-family:arial,sans-serif">ARM::BI__builtin_neon_vaddhn_v<br>
</div><div style="font-family:arial,sans-serif">ARM::BI__builtin_neon_vsubhn_v<br></div><div style="font-family:arial,sans-serif">ARM::BI__builtin_neon_vqdmlal_v<br></div><div style="font-family:arial,sans-serif">ARM::BI__builtin_neon_vqdmlsl_v</div>
</div><div style="font-family:arial,sans-serif"><br></div><div>4) Defined some more PatFrag to simplify the patterns being matched.</div><div>5) Some test cases are also modified accordingly.</div><div><br></div><div>Thanks,</div>
<div>-Jiangning</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/8/27 Tim Northover <span dir="ltr"><<a href="mailto:tnorthover@apple.com" target="_blank">tnorthover@apple.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">> I think essentially you are intending to remove some of the builtins from 32-bit ARM target.<br>
<br>
</div>Yep, that's correct. And your list matches with my plans for now too.<br>
<div class="im"><br>
> Do you mean to use lshr instead of shl, because addhn is to get high narrow, so we could have,<br>
<br>
</div>Yes, I did sorry. I'd discovered this, and was planning to mention it when posting the patches.<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>Thanks,<div>-Jiangning</div>
</div>