From 38a47d0db0931a870d0044c20d7aca68c19ded38 Mon Sep 17 00:00:00 2001 From: Hans Wennborg Date: Mon, 14 Dec 2015 18:46:11 +0000 Subject: [PATCH] clang-cl: make /Wall turn on both -Wall and -Wextra (PR25563) The documentation suggests /Wall should really turn on -Wextra and any other warnings that are not enabled by default. That would correspond to Clang's -Weverything, but is probably not what users want. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@255524 91177308-0d34-0410-b5e6-96231b3b80d8 --- include/clang/Driver/CLCompatOptions.td | 2 +- test/Driver/cl-options.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/include/clang/Driver/CLCompatOptions.td b/include/clang/Driver/CLCompatOptions.td index 8da6e8c08c..d5a3ff8f14 100644 --- a/include/clang/Driver/CLCompatOptions.td +++ b/include/clang/Driver/CLCompatOptions.td @@ -120,7 +120,7 @@ def _SLASH_W1 : CLFlag<"W1">, HelpText<"Enable -Wall">, Alias; def _SLASH_W2 : CLFlag<"W2">, HelpText<"Enable -Wall">, Alias; def _SLASH_W3 : CLFlag<"W3">, HelpText<"Enable -Wall">, Alias; def _SLASH_W4 : CLFlag<"W4">, HelpText<"Enable -Wall and -Wextra">, Alias; -def _SLASH_Wall : CLFlag<"Wall">, HelpText<"Enable -Wall">, Alias; +def _SLASH_Wall : CLFlag<"Wall">, HelpText<"Enable -Wall and -Wextra">, Alias; def _SLASH_WX : CLFlag<"WX">, HelpText<"Treat warnings as errors">, Alias, AliasArgs<["error"]>; def _SLASH_WX_ : CLFlag<"WX-">, HelpText<"Do not treat warnings as errors">, diff --git a/test/Driver/cl-options.c b/test/Driver/cl-options.c index fc56be8993..c75ec9504f 100644 --- a/test/Driver/cl-options.c +++ b/test/Driver/cl-options.c @@ -174,7 +174,7 @@ // RUN: %clang_cl /W2 -### -- %s 2>&1 | FileCheck -check-prefix=W1 %s // RUN: %clang_cl /W3 -### -- %s 2>&1 | FileCheck -check-prefix=W1 %s // RUN: %clang_cl /W4 -### -- %s 2>&1 | FileCheck -check-prefix=W4 %s -// RUN: %clang_cl /Wall -### -- %s 2>&1 | FileCheck -check-prefix=W1 %s +// RUN: %clang_cl /Wall -### -- %s 2>&1 | FileCheck -check-prefix=W4 %s // W1: -Wall // W4: -WCL4 -- 2.40.0