<div dir="ltr">I don't think there is disagreement about the direction you're going with this (having a single point of truth for these definitions), and 1-7 seem to categorize as "obvious" in view of that direction, so feel free to commit (as incremental patches, of course).<div><br></div><div>Any issues in small patches like this can be figured out in post-commit (i.e. as opposed to doing pre-commit review on phabricator) as long as there is agreement about the general direction. The major need for pre-commit review is patches that are starting in a new direction (for major changes, RFC on llvm-dev is appropriate) and for especially large patches that introduce significant new functionality. (there is some more information here: <a href="http://llvm.org/docs/DeveloperPolicy.html#incremental-development">http://llvm.org/docs/DeveloperPolicy.html#incremental-development</a> )</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 20, 2015 at 1:39 PM, Xinliang David Li <span dir="ltr"><<a href="mailto:davidxl@google.com" target="_blank">davidxl@google.com</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">Those changes are highly related and most of the changes are documentation and in one file only. I think it is better to review the changes in one piece (for better context).<div><br></div><div>Later I can of course commit the reviewed changes in smaller pieces (logically just two of them).</div><div><br></div><div>thanks,</div><div><br></div><div>David<br><div><br></div></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 19, 2015 at 7:28 PM, Sean Silva <span dir="ltr"><<a href="mailto:chisophugis@gmail.com" target="_blank">chisophugis@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">silvas added a subscriber: silvas.<br>
silvas requested changes to this revision.<br>
silvas added a reviewer: silvas.<br>
silvas added a comment.<br>
This revision now requires changes to proceed.<br>
<br>
Your 1-7 are natural separate patches, so please split this patch up.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D14854" rel="noreferrer" target="_blank">http://reviews.llvm.org/D14854</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>