[PATCH] D114651: [clang-cl] Expose -Wall to clang-cl by unaliasing -Wall, keeping /Wall as alias to -Weverything
Martin Storsjö via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 29 12:20:53 PST 2021
mstorsjo added a comment.
In D114651#3158807 <https://reviews.llvm.org/D114651#3158807>, @rnk wrote:
> I will add that multiple users have run into this problem, and I think it might be more practical to consider unaliasing Wall altogether. Clang doesn't emit the same set of warnings as MSVC. Anyone seriously using `clang-cl /Wall` is going to receive a pile of `-WcxxNN-compat` warnings that are self-contradictory, and they are going to need to curate a set of clang-specific warning flags anyway.
One possible downside with that, is that you'd easily end up making cl-compatible build files that works nicely with clang-cl but spew warnings with cl.exe. (Although maybe cl.exe's -Wall produce a much safer/saner set of warnings?) But I don't feel strongly about it..
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits