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