<div dir="ltr">On Wed, May 29, 2013 at 6:03 PM, Edwin Vane <span dir="ltr"><<a href="mailto:edwin.vane@intel.com" target="_blank">edwin.vane@intel.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  Moving to post-commit review.<br></blockquote><div><br></div><div style>Just FYI: this is usually a discouraged pattern :) If you have the feeling that something needs pre-commit review once, there are very few things that would make it actually useful to switch to post-commit (apart from doug telling you on IRC, which you would then note in the review)</div>
<div style><br></div><div style>Cheers,</div><div style>/Manuel</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<a href="http://llvm-reviews.chandlerc.com/D882" target="_blank">http://llvm-reviews.chandlerc.com/D882</a><br>
<br>
BRANCH<br>
  callbacks<br>
<br>
ARCANIST PROJECT<br>
  clang<br>
</blockquote></div><br></div></div>