<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Nov 13, 2013 at 4:03 AM, Daniel Sanders <span dir="ltr"><<a href="mailto:daniel.sanders@imgtec.com" target="_blank">daniel.sanders@imgtec.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  Can I check a matter of policy?<br>
<br>
  Is it ok to commit the version of the patch you LGTM'd and then fix the MIPS-specific false passes (caused by the missing 'virtual' keyword) under the 'obvious change' rule? or should I wait for an explicit LGTM on the updated patch?<br>
</blockquote><div><br></div><div>Generally if someone says "LGTM as long as you do blah" you're entrusted to make those changes in your current patch and commit in one go without further review. If for some reason you got the requested changes wrong, they'll generally be addressed in post-commit review (which, if the wrongness was sufficiently severe, could include reverting, but that's usually uncommon/unlikely as such extra changes are usually simple/mechanical in nature)</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
  I don't mind either way. I just want to double-check what I should be doing for this patch.<br>
<br>
  Thanks<br>
<div class="HOEnZb"><div class="h5"><br>
<a href="http://llvm-reviews.chandlerc.com/D1973" target="_blank">http://llvm-reviews.chandlerc.com/D1973</a><br>
<br>
BRANCH<br>
  msa/bugfix-getconstant2<br>
<br>
ARCANIST PROJECT<br>
  llvm<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div></div>