<div dir="ltr">eh, I think that's probably the wrong direction for LLVM, actually - I think we've generally encouraged "const" being explicit when it's otherwise wrapped up in "auto" - same as for *. </div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Apr 19, 2021 at 11:12 AM Nathan James 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">For the record the warning about turning `auto *X` into `const auto *X`<br>
shouldn't be emitted, The check was adapted so that warning should no<br>
longer be emitted in llvm code. <br>
<a href="https://github.com/llvm/llvm-project/commit/8a68c40a1bf256523993ee97b39f79001eaade91" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/8a68c40a1bf256523993ee97b39f79001eaade91</a><br>
<br>
I can only guess that the pre-merge build bot is using an old build of<br>
clang-tidy as that commit should be in the 11.0.0 release.<br>
<br>
~Nathan James<br>
<br>
On Mon, 2021-04-19 at 09:43 +0000, Maxim Kazantsev via llvm-dev wrote:<br>
> Hello everyone,<br>
>  <br>
> I started noticing that lately we’ve improved reporting from clang-<br>
> tidy, pointing out at various formatting issues. However the more<br>
> verbose it becomes, the more annoyed I feel about it. For example<br>
> here:<br>
>  <br>
> <a href="https://reviews.llvm.org/D100721" rel="noreferrer" target="_blank">https://reviews.llvm.org/D100721</a><br>
>  <br>
> It complains literally about every second line, inserting its<br>
> comments straight into review. They take as much space as the actual<br>
> code. Maybe it’s just me, but it’s really hard to me to understand<br>
> what the patch is actually doing with so many inlined auto-generated<br>
> comments. Maybe there is a button to hide them somewhere, but I<br>
> failed to find it.<br>
>  <br>
> I understand what was the intention, and clang-tidy is a cool thing<br>
> in general, but it’s getting too intrusive. Does anyone else have the<br>
> same problem as I do? If there’s a lot of people whom it annoys,<br>
> maybe we should think how to make it less invasive. Maybe it should<br>
> put these comment when the patch gets approved, or something like<br>
> this.<br>
>  <br>
> Thanks,<br>
> Max<br>
>  <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>
<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>