<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 22, 2014 at 5:15 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank" class="cremed">dblaikie@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="">On Thu, May 22, 2014 at 4:54 AM, Daniel Jasper <<a href="mailto:djasper@google.com" class="cremed">djasper@google.com</a>> wrote:<br>

> As there was no objection by now, I assume this is acceptable.<br>
<br>
</div>It's generally considered poor form to submit code you've sent review<br>
when you haven't received that review... it creates a difficult<br>
situation where people may feel it acceptable to submit due to long<br>
review turnaround which isn't something we want to encourage in the<br>
community.<br></blockquote><div><br></div><div>And you think me and the reviewer sitting right next to me can't figure this out ...</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

If you send something for review, it's presumed to be because you<br>
don't feel confident that it's the right change and need a second set<br>
of eyes to confirm/discuss that with - time shouldn't change that.<br>
<br>
- David<br>
<br>
><br>
> <a href="http://reviews.llvm.org/D418" target="_blank" class="cremed">http://reviews.llvm.org/D418</a><br>
><br>
><br>
><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu" class="cremed">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank" class="cremed">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br></div></div>