<div dir="ltr">Hi Chad,<div><br></div><div>I'm happy with this, and your new patch looks good to me if only you can add more comments in the code around excluding AND, because at the first look it's strange.</div><div>
<br></div><div>Thanks,<br>-Jiangning</div><div><br></div><div class="gmail_extra"><div class="gmail_quote">2014-07-30 0:02 GMT+08:00 Chad Rosier <span dir="ltr"><<a href="mailto:mcrosier@codeaurora.org" target="_blank">mcrosier@codeaurora.org</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">>>! In D4440#12, @mcrosier wrote:<br>
> The performance numbers are slightly better, but still less than noise.  All, please take a look.<br>
<br>
</div>To be clear, the numbers are slightly better relative to the ADD/SUB only combine.  Overall, we still see a large improvement in eembc/OAv2.<br>
<br>
<a href="http://reviews.llvm.org/D4440" target="_blank">http://reviews.llvm.org/D4440</a><br>
<br>
<br>
</blockquote></div><br></div></div>