<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 18, 2015 at 7:41 AM, Daniel Marjamäki <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">danielmarjamaki marked an inline comment as done.<br>
danielmarjamaki added a comment.<br>
<br>
Does anybody else think that this should be moved to clang-tidy?<br></blockquote><div><br>Yep<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I believe that the noise level is very low when I scan various open source projects.</blockquote><div><br></div><div>depends how you define noise, though - and the Clang project has historically (& Richard's comments indicate that philosophy is ongoing) treated anything that didn't indicate a bug or bug-like issue (where bug is "this program behaves differently from the way the programmer intended it to") as noise, so every instance of this diagnostic is noise by that definition, I believe (the suggested changes, by design, never change the behavior of the program)</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> My script tries to run clang on all projects in the debian archive. I have shown triaged results before, but if you are not convinced I can upload the complete results so you can look at that. For comparison, I get more noise from existing compiler warnings.<br>
<br>
<br>
================<br>
Comment at: include/clang/AST/Decl.h:791<br>
@@ +790,3 @@<br>
+<br>
<span class="">+    /// \brief Whether this variable has non-const use so it can't be const.<br>
+    unsigned NonConstUse:1;<br>
</span>----------------<br>
hmm.. I'll change "variable" to "parameter" in the next diff.<br>
<br>
================<br>
Comment at: include/clang/AST/Decl.h:857<br>
@@ -853,3 +856,3 @@<br>
 public:<br>
   typedef redeclarable_base::redecl_range redecl_range;<br>
   typedef redeclarable_base::redecl_iterator redecl_iterator;<br>
----------------<br>
Thanks! I have moved NonConstUse to ParmVarDeclBits.<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="http://reviews.llvm.org/D12359" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12359</a><br>
<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div></div>