]> granicus.if.org Git - clang/commitdiff
InitializePreprocessor cannot fail.
authorDaniel Dunbar <daniel@zuster.org>
Wed, 4 Nov 2009 21:13:15 +0000 (21:13 +0000)
committerDaniel Dunbar <daniel@zuster.org>
Wed, 4 Nov 2009 21:13:15 +0000 (21:13 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@86048 91177308-0d34-0410-b5e6-96231b3b80d8

include/clang/Frontend/InitPreprocessor.h
lib/Frontend/InitPreprocessor.cpp
tools/clang-cc/clang-cc.cpp

index bf27b19aa3d7f14ee37b5b8b751efcc0e14d35e0..415acea79a26c95d09f65a1019c3bdee3fed83bd 100644 (file)
@@ -69,9 +69,9 @@ public:
 };
 
 /// InitializePreprocessor - Initialize the preprocessor getting it and the
-/// environment ready to process a single file. This returns true on error.
+/// environment ready to process a single file.
 ///
-bool InitializePreprocessor(Preprocessor &PP,
+void InitializePreprocessor(Preprocessor &PP,
                             const PreprocessorInitOptions& InitOptions);
 
 } // end namespace clang
index 3a54d1dca78096c3accf2e0be0a1aac309a3543d..7139e55f0b6009f77af60c26163ee077a8b7208f 100644 (file)
@@ -442,7 +442,7 @@ static void InitializePredefinedMacros(const TargetInfo &TI,
 /// InitializePreprocessor - Initialize the preprocessor getting it and the
 /// environment ready to process a single file. This returns true on error.
 ///
-bool clang::InitializePreprocessor(Preprocessor &PP,
+void clang::InitializePreprocessor(Preprocessor &PP,
                                    const PreprocessorInitOptions &InitOpts) {
   std::vector<char> PredefineBuffer;
 
@@ -488,7 +488,4 @@ bool clang::InitializePreprocessor(Preprocessor &PP,
   // Null terminate PredefinedBuffer and add it.
   PredefineBuffer.push_back(0);
   PP.setPredefines(&PredefineBuffer[0]);
-
-  // Once we've read this, we're done.
-  return false;
 }
index 66aa19a5d314ef93381efd0066c676fd5d6bd56d..95f63704014183fab58adf3d9652f19f1821bdcf 100644 (file)
@@ -1251,8 +1251,7 @@ public:
 
     PreprocessorInitOptions InitOpts;
     InitializePreprocessorInitOptions(InitOpts);
-    if (InitializePreprocessor(*PP, InitOpts))
-      return 0;
+    InitializePreprocessor(*PP, InitOpts);
 
     return PP.take();
   }