<div dir="ltr">Hi Eric,<div><br></div><div>OK. Attached are two separate patches.</div><div><br></div><div>Hi Qunentin,</div><div><br></div><div>I modified the code following your new comments.</div><div><br></div><div>Since I don't know how phabaricator will work for two patches uploaded, I'm simply reply this mail by attached two separate patches. I don't want to create a new review entry in phabaricator to confuse the people who want to review the patch.</div>
<div><br></div><div>Thanks,<br></div><div>-Jiangning</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-07-08 5:25 GMT+08:00 Eric Christopher <span dir="ltr"><<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">You could also split the patch into adding the API and using it in the next commit. I.e. there's no reason why isRematerializable can't be set for the instructions separate from adding TII->isCheapAsAMove()<br>

<br>
<a href="http://reviews.llvm.org/D4361" target="_blank">http://reviews.llvm.org/D4361</a><br>
<br>
<br>
</blockquote></div><br></div>