<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">2014-04-06 20:32 GMT+07:00 Erik Verbruggen <span dir="ltr"><<a href="mailto:erik.verbruggen@me.com" target="_blank">erik.verbruggen@me.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><br>
  Could you rename the method to ProcessUMulZExtIdiom? The ProcessUAddIdiom handles a different case, and I have a patch that does the same for UMul.<br>
<br>
  Otherwise, LGTM, but bkramer has the final say.<br></blockquote><div><br></div>Thank you for review. I updated fix in phabricator. Waiting for bkramer's word.<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<br>
  As a follow-up, are you interested in doing the same for a signed sub and mul? (Add is already handled.) If not, I can do that.<br>
<br></blockquote><div><br></div><div>Yes, please do that. I took these old problem in hope I wouldn't interfere with anybody. I am sorry I hindered you.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<a href="http://reviews.llvm.org/D2814" target="_blank">http://reviews.llvm.org/D2814</a><br>
<br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Thanks,<br>--Serge<br>
</div></div>