<div dir="ltr">I was thinking of `-Wreturn-std-move`, which is -Wmove/-Wmost/-Wall but not always-on.<div>Grepping the code for DefaultIgnore, I see that -Wfor-loop-analysis is another example (but 4 years old).</div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Jan 8, 2019 at 2:37 PM John McCall <<a href="mailto:jmccall@apple.com">jmccall@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><br>
<br>
On 8 Jan 2019, at 13:45, Arthur O'Dwyer wrote:<br>
<br>
> On Tue, Jan 8, 2019 at 1:05 PM John McCall via cfe-dev <<br>
> <a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>> wrote:<br>
><br>
>> On 8 Jan 2019, at 8:37, Arnaud Bienner wrote:<br>
>>><br>
>>> I realized I didn't put "DefaultIgnore" on this warning.<br>
>>> I'm not experienced enough with clang to know what's the best way to <br>
>>> deal<br>
>>> with new warnings, but my feeling is that it would be sensible to <br>
>>> have<br>
>> this<br>
>>> new warning DefaultIgnore for now, in -Wall, and make it default <br>
>>> once we<br>
>>> have some feedback from the community: while not all C++ projects <br>
>>> use<br>
>> -Wall<br>
>>> (or -Wextra) I believe enough do to give us a chance to get some<br>
>> feedback.<br>
>>><br>
>>> What do you think?<br>
>><br>
>> We generally don't add things to -Wall.  That's why I went into my <br>
>> whole<br>
>> spiel<br>
>> about versioning: I think it's a conversation we need to have before <br>
>> we're<br>
>> ready to accept this as a warning that's anything but hidden <br>
>> permanently<br>
>> behind its own opt-in flag.<br>
>><br>
><br>
> John: Wha? Clang *frequently* adds things to -Wall!  -Wall includes <br>
> -Wmost<br>
> which includes a bunch of other categories, so while we don't often <br>
> put new<br>
> diagnostics *directly* under -Wall, pretty much every "reasonable"<br>
> diagnostic eventually winds up in there somehow — which is the <br>
> intent.<br>
<br>
I don't think we "frequently" add things to -Wall or -Wmost.  We do <br>
somewhat<br>
frequently add warnings that are unconditionally default-on, but the <br>
groups<br>
have a conventional meaning that we don't generally touch.  What <br>
recently-added<br>
warnings are you thinking of that are not default-on but which are <br>
included<br>
in a group like -Wall or -Wmost?<br>
<br>
John.<br>
</blockquote></div>