<div dir="ltr"><div dir="ltr"><a class="gmail_plusreply" id="plusReplyChip-7" href="mailto:goncharov@google.com" tabindex="-1">+Mikhail Goncharov</a> <br><div><br></div>Hi Fangrui,</div><div dir="ltr"><br><div>I'm glad you like the tool!</div><div><br></div><div>We got a bunch of issues reported around clang-tidy, so Mikhail disabled it today. Since the thread has a few other points, I created a separate issue [1] for it.</div><div><br></div><div>The linter script [2] is already using clang-tidy-diff on the git diff. So it should only look into changed lines... </div><div><br></div><div>[1] <a href="https://github.com/google/llvm-premerge-checks/issues/94">https://github.com/google/llvm-premerge-checks/issues/94</a></div><div>[2] <a href="https://github.com/google/llvm-premerge-checks/blob/master/scripts/lint.sh">https://github.com/google/llvm-premerge-checks/blob/master/scripts/lint.sh</a></div><div><br></div><div>Best,</div><div>Christian</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Jan 4, 2020 at 7:59 PM Mark de Wever via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Fri, Jan 03, 2020 at 04:36:25PM +0000, Doerfert, Johannes via llvm-dev wrote:<br>
> I second Fangrui's findings and add:<br>
> <br>
> - clang-format should only run on "source" files. For one, it runs on<br>
>   non-C/C++ files right now with the results you can imagine. Even if it<br>
>   is a C/C++ test we do not always want to format it.<br>
<br>
I already file a bug report to request not to format the tests [1].<br>
<br>
[1] <a href="https://github.com/google/llvm-premerge-checks/issues/89" rel="noreferrer" target="_blank">https://github.com/google/llvm-premerge-checks/issues/89</a><br>
<br>
<br>
Kind regards,<br>
Mark de Wever<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr">Best,<div>Christian</div></div></div></div>