[PATCH] D65706: [docs] Better documentation for -Weverything

JF Bastien via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Aug 4 20:37:57 PDT 2019


jfb added inline comments.


================
Comment at: clang/docs/UsersManual.rst:999-1000
+diagnostics contradict each other, users of :option:`-Weverything` therefore
+often disable many diagnostics such as :option:`-Wno-c++98-compat`
+:option:`-Wno-c++-compat`.
+
----------------
aaron.ballman wrote:
> Would you care to propose a more exhaustive list of conflicting diagnostics? (Perhaps in a follow-up patch.)
I looked a bit and I'm worried that providing a list won't be particularly satisfying for people looking at this. I think it's better to have some warning, and let folks figure out what works for their particular situation. Here I'm assuming that they don't use C++98 and that seems reasonable, but figuring out what side of contradictions they're on doesn't seem like it'll work out.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65706/new/

https://reviews.llvm.org/D65706





More information about the cfe-commits mailing list