]> granicus.if.org Git - clang/commitdiff
Simplify ProcessWarningOptions since it can't fail
authorKovarththanan Rajaratnam <kovarththanan.rajaratnam@gmail.com>
Wed, 17 Mar 2010 09:36:02 +0000 (09:36 +0000)
committerKovarththanan Rajaratnam <kovarththanan.rajaratnam@gmail.com>
Wed, 17 Mar 2010 09:36:02 +0000 (09:36 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@98721 91177308-0d34-0410-b5e6-96231b3b80d8

include/clang/Frontend/Utils.h
lib/Frontend/CompilerInstance.cpp
lib/Frontend/Warnings.cpp

index 7f43b2ad80ff4809c32a40a18d1c4f637c39f451..c1d483164a21bb731fd675be74bbe0e2de02dfbd 100644 (file)
@@ -59,7 +59,7 @@ void InitializePreprocessor(Preprocessor &PP,
 
 /// ProcessWarningOptions - Initialize the diagnostic client and process the
 /// warning options specified on the command line.
-bool ProcessWarningOptions(Diagnostic &Diags, const DiagnosticOptions &Opts);
+void ProcessWarningOptions(Diagnostic &Diags, const DiagnosticOptions &Opts);
 
 /// DoPrintPreprocessedInput - Implement -E mode.
 void DoPrintPreprocessedInput(Preprocessor &PP, llvm::raw_ostream* OS,
index 2f7c2481f6d93ebd3ecb3e19dd31d78960088ecd..edd19d7a539bf77ac2fdf31ed4aa7aa2ecd4146a 100644 (file)
@@ -169,8 +169,7 @@ Diagnostic *CompilerInstance::createDiagnostics(const DiagnosticOptions &Opts,
     SetUpBuildDumpLog(Opts, Argc, Argv, *Diags);
 
   // Configure our handling of diagnostics.
-  if (ProcessWarningOptions(*Diags, Opts))
-    return 0;
+  ProcessWarningOptions(*Diags, Opts);
 
   return Diags.take();
 }
index 4bf507d48dca550300a6bee8c20709cac86d63ef..ea9635e79849a3e298e447c6dc3c0cba06664746 100644 (file)
@@ -31,7 +31,7 @@
 #include <algorithm>
 using namespace clang;
 
-bool clang::ProcessWarningOptions(Diagnostic &Diags,
+void clang::ProcessWarningOptions(Diagnostic &Diags,
                                   const DiagnosticOptions &Opts) {
   Diags.setSuppressSystemWarnings(true);  // Default to -Wno-system-headers
   Diags.setIgnoreAllWarnings(Opts.IgnoreWarnings);
@@ -122,6 +122,4 @@ bool clang::ProcessWarningOptions(Diagnostic &Diags,
     if (Diags.setDiagnosticGroupMapping(OptStart, Mapping))
       Diags.Report(diag::warn_unknown_warning_option) << ("-W" + Opt);
   }
-
-  return false;
 }