<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, May 23, 2014 at 12:36 AM, Andrey Bokhanko <span dir="ltr"><<a href="mailto:andreybokhanko@gmail.com" target="_blank">andreybokhanko@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div>Also, Chandler, is it safe to assume that if some time (a week? two weeks?) passed and no-one but Chris (or someone else knowledgeable with OpenMP but not clang code owner) reviewed a patch, the very fact of no review [from clang code owners] suggests that the patch is not disruptive enough / too OpenMP-specific for clang code owners to step in and do code review personally? Especially given that said "clang code owners" are extremely few in numbers -- like three people on Earth, if I remember correctly?<br>
</div></blockquote><div><br></div><div><a href="http://llvm.org/docs/DeveloperPolicy.html#code-reviews">http://llvm.org/docs/DeveloperPolicy.html#code-reviews</a> lays out pretty clearly that no, it isn't OK?</div><div>
<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div>Granted, there are areas where a review from code owner is a *must*. But in my opinion, such areas are quite limited in case of OpenMP patches -- basically, only when common infrastructure / code is changed. Agree?<br>
</div></blockquote></div><br>I think that the code owners will let you know when you have patches that could have gone in with post-commit review. Certainly, in areas where I am a frequent reviewer I always make sure to note in reviews when I think the patch satisfied the "obviousness" bar or was sufficiently isolated to a part of the code maintained by the person already.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">I cannot possibly say whether or not this is the case -- I don't actually know these parts of Clang well enough. If you get Doug, or John, or Richard to say so, you're good-to-go. I suspect they would do so in the code review itself.</div>
</div>