[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]

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..


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114651