<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Mar 7, 2016 at 1:40 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>
<span class=""><br>
In <a href="http://reviews.llvm.org/D16381#367348" rel="noreferrer" target="_blank">http://reviews.llvm.org/D16381#367348</a>, @davidxl wrote:<br>
<br>
> LGTM.<br>
><br>
> This looks really great, so let's move on with this long waited missing feature.<br>
<br>
<br>
</span>David, this is not an area of LLVM you have done substantial work on, and this is a very significant feature.<br></blockquote><div><br></div><div>The patch has been reviewed and tested thoroughly. The approach used by the patch has been discussed many times in the past since last October in various contexts in which you were involved. </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I'm sorry that I have not had time to review this yet, but the correct response is not for you to make a patch as LGTM.</blockquote><div><br></div><div><br></div><div>Again we have done extensive review on this patch. According to your standard, it looks like only you can approve patches in the inliner?  IMO this is not healthy to the project and more people need to get involved. As it stands today, it seems to be very hard to everybody in the community to make any changes in the inliner due to this. </div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> Please revert this and let's actually get it reviewed before it goes into the project.<br>
<div class="HOEnZb"><div class="h5"><br></div></div></blockquote><div><br></div><div>Actually get reviewed?  What do you mean? Have you really followed the thread?   If you want to be constructive, please do your part of the review, but not request a revert like this.</div><div><br></div><div>David</div><div><br></div><div><br></div><div><br></div><div><br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D16381" rel="noreferrer" target="_blank">http://reviews.llvm.org/D16381</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br></div></div>