[PATCH] D114651: [clang-cl] Expose -Wall to clang-cl by unaliasing -Wall, keeping /Wall as alias to -Weverything

Markus Böck via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 26 11:29:08 PST 2021


zero9178 added a comment.

Is the deviation from MSVC behaviour here intentional? MSVC flags allow both using a `/` as well as `-` as prefix. That means Both `-Wall` and `/Wall` are accepted by MSVC as well as clang-cl and in both compilers currently lead to ALL warnings being emitted. So this patch would deviate from that behaviour as well as add confusion as every other option behaves the same in MSVC and clang-cl, regardless of whether `-` or `/` is used as prefix.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114651



More information about the cfe-commits mailing list