<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Sep 7, 2014 at 11:23 PM, Daniel Jasper <span dir="ltr"><<a href="mailto:djasper@google.com" target="_blank">djasper@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 id=":3a4" class="a3s" style="overflow:hidden">Do we really need a separate file for Clang? Shouldn't D5236 suffice as Clang is always checked out into LLVM?<br></div></blockquote><div><br></div><div>There are folks that still develop without Clang checked out under LLVM. Seems cheap to support that here.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":3a4" class="a3s" style="overflow:hidden">
<br>
On the other hand, I would add files containing "Checks: '-*'" into the test/ directories of both. Mosts test won't be (and don't need to be) clang-tidy clean.</div></blockquote></div><div class="gmail_extra"><br></div>Agreed.<br><br></div></div>