<div dir="ltr"><div>This breaks the sphinx bot:</div><div><br></div><div><a href="http://lab.llvm.org:8011/builders/clang-sphinx-docs/builds/45204/steps/docs-clang-html/logs/stdio">http://lab.llvm.org:8011/builders/clang-sphinx-docs/builds/45204/steps/docs-clang-html/logs/stdio</a><br></div><div><br></div>Warning, treated as error:<br>/home/buildbot/llvm-build-dir/clang-sphinx-docs/llvm/src/tools/clang/docs/UsersManual.rst:995: WARNING: unknown option: -Wno-c++98-compat<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Aug 5, 2019 at 12:52 PM JF Bastien via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@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">Author: jfb<br>
Date: Mon Aug  5 09:53:45 2019<br>
New Revision: 367889<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=367889&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=367889&view=rev</a><br>
Log:<br>
[docs] document -Weveything more betterer<br>
<br>
Reviewers: aaron.ballman<br>
<br>
Subscribers: jkorous, dexonsmith, cfe-commits<br>
<br>
Tags: #clang<br>
<br>
Differential Revision: <a href="https://reviews.llvm.org/D65706" rel="noreferrer" target="_blank">https://reviews.llvm.org/D65706</a><br>
<br>
Modified:<br>
    cfe/trunk/docs/UsersManual.rst<br>
<br>
Modified: cfe/trunk/docs/UsersManual.rst<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/docs/UsersManual.rst?rev=367889&r1=367888&r2=367889&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/cfe/trunk/docs/UsersManual.rst?rev=367889&r1=367888&r2=367889&view=diff</a><br>
==============================================================================<br>
--- cfe/trunk/docs/UsersManual.rst (original)<br>
+++ cfe/trunk/docs/UsersManual.rst Mon Aug  5 09:53:45 2019<br>
@@ -992,13 +992,24 @@ is treated as a system header.<br>
 Enabling All Diagnostics<br>
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^<br>
<br>
-In addition to the traditional ``-W`` flags, one can enable **all**<br>
-diagnostics by passing :option:`-Weverything`. This works as expected<br>
-with<br>
-:option:`-Werror`, and also includes the warnings from :option:`-pedantic`.<br>
+In addition to the traditional ``-W`` flags, one can enable **all** diagnostics<br>
+by passing :option:`-Weverything`. This works as expected with<br>
+:option:`-Werror`, and also includes the warnings from :option:`-pedantic`. Some<br>
+diagnostics contradict each other, therefore, users of :option:`-Weverything`<br>
+often disable many diagnostics such as :option:`-Wno-c++98-compat`<br>
+:option:`-Wno-c++-compat` because they contradict recent C++ standards.<br>
<br>
-Note that when combined with :option:`-w` (which disables all warnings), that<br>
-flag wins.<br>
+Since :option:`-Weverything` enables every diagnostic, we generally don't<br>
+recommend using it. :option:`-Wall` :option:`-Wextra` are a better choice for<br>
+most projects. Using :option:`-Weverything` means that updating your compiler is<br>
+more difficult because you're exposed to experimental diagnostics which might be<br>
+of lower quality than the default ones. If you do use :option:`-Weverything`<br>
+then we advise that you address all new compiler diagnostics as they get added<br>
+to Clang, either by fixing everything they find or explicitly disabling that<br>
+diagnostic with its corresponding `Wno-` option.<br>
+<br>
+Note that when combined with :option:`-w` (which disables all warnings),<br>
+disabling all warnings wins.<br>
<br>
 Controlling Static Analyzer Diagnostics<br>
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</blockquote></div>