<div dir="ltr">rnk is OOO until tomorrow btw.</div><br><div class="gmail_quote"><div dir="ltr">On Mon, Oct 10, 2016 at 1:00 AM Manuel Klimek <<a href="mailto:klimek@google.com">klimek@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">klimek added a comment.<br class="gmail_msg">
<br class="gmail_msg">
Drive-by comment regarding the process of creating changes like this:<br class="gmail_msg">
In our somewhat larger codebase we've found that the best way to create this type of CL is by writing a clang tool:<br class="gmail_msg">
<br class="gmail_msg">
- the clang tool can be reviewed on its own, and behavior can be bikeshed on the tests<br class="gmail_msg">
- it's easy to re-run a clang tool on a subset of the interfaces to produce any size CL you want<br class="gmail_msg">
- if you can make it (or convert it into) a clang-tidy check with a fixit, we prevent regressions<br class="gmail_msg">
<br class="gmail_msg">
That said, if others are fine reviewing this change I'm fine with it; this is mainly an idea for the future :)<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D25257" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D25257</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>