Ugh this is silly… fixed again. > On Aug 5, 2019, at 12:55 PM, Nico Weber <tha...@chromium.org> wrote: > > Still sad, now with a different message: > http://lab.llvm.org:8011/builders/clang-sphinx-docs/builds/45220 > <http://lab.llvm.org:8011/builders/clang-sphinx-docs/builds/45220> > > /home/buildbot/llvm-build-dir/clang-sphinx-docs/llvm/src/tools/clang/docs/UsersManual.rst:1002: > WARNING: unknown option: -Wall > > On Mon, Aug 5, 2019 at 3:46 PM JF Bastien <jfbast...@apple.com > <mailto:jfbast...@apple.com>> wrote: > Fixed. I guess we should document those... > >> On Aug 5, 2019, at 10:51 AM, Nico Weber <tha...@chromium.org >> <mailto: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
