<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Oct 26, 2013 at 11:53 PM, Shuxin Yang <span dir="ltr"><<a href="mailto:shuxin.llvm@gmail.com" target="_blank" class="cremed">shuxin.llvm@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Also, while I am sure this is the right approach, I made some comments on this patch:<br>

<br>
<br>
<a href="http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20131021/192186.html" target="_blank" class="cremed">http://lists.cs.uiuc.edu/<u></u>pipermail/llvm-commits/Week-<u></u>of-Mon-20131021/192186.html</a> <br>

<br>
it looks like you committed without replying or acting on any of my review comments? Huh?<br>
<br>
</blockquote></div>
I somehow miss this mail.  There is no reason to ignore any response. I have never ever<br>
ignore feedback before. Actually I'm expecting your feedback as you are original author of the globalopt.</blockquote></div><br>Just a note to help future code reviews Shuxin:</div><div class="gmail_extra"><br></div>
<div class="gmail_extra">It feels like you're approaching the review a bit wrong. Missing an email with further comments shouldn't lead you to commit anything. The idea is that once you start code review, you're waiting for a reviewer to give an explicit 'LGTM' before you commit.</div>
</div>