<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 8, 2016 at 10:41 AM, Xinliang David Li via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Tue, Mar 8, 2016 at 4:20 AM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">chandlerc added a comment.<br>
<br>
First and foremost, sorry about snapping earlier. I shouldn't have done that, I was frustrated and not communicating very effectively. Thanks to Sean and Hal and others who wrote constructive and helpful emails to get this back on the rails. Secondly, sorry that I've neglected this patch for so long. I kept prioritizing working on the actual pass manager stuff over it, and I should have at least written this up.<br></blockquote><div><br></div></span><div>Apology accepted. Chandler, as a long time contributor to LLVM project, new contributors will see you as an example to follow. I don't see the following match what people expect to see: Ignoring review requests, showing disagreement by keeping silence, implying people not respecting your authority in your area, or dictating who can/can not do this or that. Reviews takes a lot of time and energy so you should encourage more people participate in it by helping people gain more experience and feel more confident to approve and get things done.   In this particular case, if you had raised any concerns in the long thread, the approval would have *not* been given without consensus being made. Even after the commit, if you had listed any concrete reasons why the patch should be reverted first for further discussion, it would have been done without a blink, but you came out with a blame first, followed by a strong demand with no reason (as if this is my domain ....)         </div><div><br></div></div></div></div></blockquote><div><br></div><div>(I will say this publicly because everything else has been done publicly at this point :P)<br><br></div><div>Guys.</div><div>I understand you are both frustrated at each other and the state of this review (and probably others)</div><div>But i don't think either of you are making useful responses to each other at this point.</div><div>It just comes off as sniping, even if it's not meant to.</div><div><br></div><div>I would suggest that we take this offline and have a chat.</div><div><br></div></div></div></div>