[cfe-commits] r138909 - in /cfe/trunk: include/clang/Basic/DiagnosticCommonKinds.td lib/Frontend/CompilerInstance.cpp test/Misc/warning-flags.c

[Douglas Gregor]

Author: dgregor
Date: Wed Aug 31 16:52:21 2011
New Revision: 138909

URL: http://llvm.org/viewvc/llvm-project?rev=138909&view=rev
Log:
Switch the "no module found" default-fatal warning to a default-fatal error.

Modified:
    cfe/trunk/include/clang/Basic/DiagnosticCommonKinds.td
    cfe/trunk/lib/Frontend/CompilerInstance.cpp
    cfe/trunk/test/Misc/warning-flags.c

Modified: cfe/trunk/include/clang/Basic/DiagnosticCommonKinds.td
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticCommonKinds.td?rev=138909&r1=138908&r2=138909&view=diff
==============================================================================
--- cfe/trunk/include/clang/Basic/DiagnosticCommonKinds.td (original)
+++ cfe/trunk/include/clang/Basic/DiagnosticCommonKinds.td Wed Aug 31 16:52:21 2011
@@ -59,7 +59,7 @@
   "cannot define a function with non-namespace scope in a friend declaration">;
 def err_deleted_non_function : Error<
   "only functions can have deleted definitions">;
-def warn_module_not_found : Warning<"module '%0' not found">, DefaultFatal;
+def err_module_not_found : Error<"module '%0' not found">, DefaultFatal;
   
 // Sema && Lex
 def ext_longlong : Extension<

Modified: cfe/trunk/lib/Frontend/CompilerInstance.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/CompilerInstance.cpp?rev=138909&r1=138908&r2=138909&view=diff
==============================================================================
--- cfe/trunk/lib/Frontend/CompilerInstance.cpp (original)
+++ cfe/trunk/lib/Frontend/CompilerInstance.cpp Wed Aug 31 16:52:21 2011
@@ -645,7 +645,7 @@
                                            /*SearchPath=*/0, 
                                            /*RelativePath=*/0);
   if (!ModuleFile) {
-    getDiagnostics().Report(ModuleNameLoc, diag::warn_module_not_found)
+    getDiagnostics().Report(ModuleNameLoc, diag::err_module_not_found)
       << ModuleName.getName()
       << SourceRange(ImportLoc, ModuleNameLoc);
     return 0;

Modified: cfe/trunk/test/Misc/warning-flags.c
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Misc/warning-flags.c?rev=138909&r1=138908&r2=138909&view=diff
==============================================================================
--- cfe/trunk/test/Misc/warning-flags.c (original)
+++ cfe/trunk/test/Misc/warning-flags.c Wed Aug 31 16:52:21 2011
@@ -17,7 +17,7 @@
 
 The list of warnings below should NEVER grow.  It should gradually shrink to 0.
 
-CHECK: Warnings without flags (312):
+CHECK: Warnings without flags (311):
 CHECK-NEXT:   auto_storage_class
 CHECK-NEXT:   backslash_newline_space
 CHECK-NEXT:   charize_microsoft_ext
@@ -230,7 +230,6 @@
 CHECK-NEXT:   warn_missing_dependent_template_keyword
 CHECK-NEXT:   warn_missing_exception_specification
 CHECK-NEXT:   warn_missing_whitespace_after_macro_name
-CHECK-NEXT:   warn_module_not_found
 CHECK-NEXT:   warn_multiple_method_decl
 CHECK-NEXT:   warn_no_constructor_for_refconst
 CHECK-NEXT:   warn_nonnull_pointers_only