diff options
author | Ben Langmuir <blangmuir@apple.com> | 2014-05-08 18:09:29 +0000 |
---|---|---|
committer | Ben Langmuir <blangmuir@apple.com> | 2014-05-08 18:09:29 +0000 |
commit | 1a04a9333b50322b95cef6d68cfef400263dbb24 (patch) | |
tree | b644343af6b1c62070f412cb4445990dba352266 | |
parent | 1dd39317b8687dbe5404ca009321eeda3331d5e6 (diff) | |
download | clang-1a04a9333b50322b95cef6d68cfef400263dbb24.tar.gz clang-1a04a9333b50322b95cef6d68cfef400263dbb24.tar.bz2 clang-1a04a9333b50322b95cef6d68cfef400263dbb24.tar.xz |
Remove -Wnon-modular-include
But keep -Wnon-modular-include-in-[framework-]module
This warning is too noisy and doesn't really indicate a problem for most
people. Even though it would only really affect people using
-Weverything, that seems bad so remove it.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@208345 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | include/clang/Basic/DiagnosticGroups.td | 2 | ||||
-rw-r--r-- | include/clang/Basic/DiagnosticLexKinds.td | 3 | ||||
-rw-r--r-- | lib/Lex/ModuleMap.cpp | 2 | ||||
-rw-r--r-- | test/Modules/require-modular-includes.m | 12 |
4 files changed, 1 insertions, 18 deletions
diff --git a/include/clang/Basic/DiagnosticGroups.td b/include/clang/Basic/DiagnosticGroups.td index a2cc7406ab..b1243cfabd 100644 --- a/include/clang/Basic/DiagnosticGroups.td +++ b/include/clang/Basic/DiagnosticGroups.td @@ -190,8 +190,6 @@ def NonModularIncludeInFrameworkModule : DiagGroup<"non-modular-include-in-framework-module">; def NonModularIncludeInModule : DiagGroup<"non-modular-include-in-module", [NonModularIncludeInFrameworkModule]>; -def NonModularInclude : DiagGroup<"non-modular-include", - [NonModularIncludeInModule]>; def InvalidNoreturn : DiagGroup<"invalid-noreturn">; def InvalidSourceEncoding : DiagGroup<"invalid-source-encoding">; def KNRPromotedParameter : DiagGroup<"knr-promoted-parameter">; diff --git a/include/clang/Basic/DiagnosticLexKinds.td b/include/clang/Basic/DiagnosticLexKinds.td index abd386fc36..6240ce0705 100644 --- a/include/clang/Basic/DiagnosticLexKinds.td +++ b/include/clang/Basic/DiagnosticLexKinds.td @@ -626,8 +626,7 @@ def warn_non_modular_include_in_framework_module : Warning< def warn_non_modular_include_in_module : Warning< "include of non-modular header inside module '%0'">, InGroup<NonModularIncludeInModule>, DefaultIgnore; -def warn_non_modular_include : Warning< - "include of non-modular header">, InGroup<NonModularInclude>, DefaultIgnore; + def warn_header_guard : Warning< "%0 is used as a header guard here, followed by #define of a different macro">, diff --git a/lib/Lex/ModuleMap.cpp b/lib/Lex/ModuleMap.cpp index f89d91d056..ce2c4f02e3 100644 --- a/lib/Lex/ModuleMap.cpp +++ b/lib/Lex/ModuleMap.cpp @@ -310,8 +310,6 @@ void ModuleMap::diagnoseHeaderInclusion(Module *RequestingModule, diag::warn_non_modular_include_in_framework_module : diag::warn_non_modular_include_in_module; Diags.Report(FilenameLoc, DiagID) << RequestingModule->getFullModuleName(); - } else { - Diags.Report(FilenameLoc, diag::warn_non_modular_include); } } diff --git a/test/Modules/require-modular-includes.m b/test/Modules/require-modular-includes.m index 55674319ff..835a352433 100644 --- a/test/Modules/require-modular-includes.m +++ b/test/Modules/require-modular-includes.m @@ -40,12 +40,6 @@ // RUN: -fmodules-cache-path=%t -I %S/Inputs/require-modular-includes \ // RUN: -Werror -fmodule-name=A -fsyntax-only -x objective-c - -// Including a non-modular header (directly) with -Wnon-modular-include -// RUN: echo '#include "NotInModule.h"' | \ -// RUN: %clang_cc1 -Wnon-modular-include -fmodules \ -// RUN: -fmodules-cache-path=%t -I %S/Inputs/require-modular-includes \ -// RUN: -fmodule-name=A -fsyntax-only -x objective-c - 2>&1 | FileCheck %s - // Including an excluded header // RUN: echo '@import IncludeExcluded;' | \ // RUN: %clang_cc1 -Wnon-modular-include-in-framework-module -fmodules \ @@ -86,10 +80,4 @@ // RUN: -fmodules-cache-path=%t -I %S/Inputs/require-modular-includes \ // RUN: -Werror -fsyntax-only -x objective-c - -// Non-framework module (fail) -// RUN: echo '@import NotFramework;' | \ -// RUN: not %clang_cc1 -Werror=non-modular-include -fmodules \ -// RUN: -fmodules-cache-path=%t -I %S/Inputs/require-modular-includes \ -// RUN: -fsyntax-only -x objective-c - 2>&1 | FileCheck %s - // CHECK: include of non-modular header |