From f81d46f45ee9d80ce13c4da81898470e2e7c2140 Mon Sep 17 00:00:00 2001 From: Ilya Biryukov Date: Wed, 21 Jun 2017 12:34:27 +0000 Subject: Fixed compiler warnings after r305890. Should fix buildbots that pass -Werror. llvm-svn: 305902 --- clang/lib/Frontend/ASTUnit.cpp | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) (limited to 'clang/lib/Frontend/ASTUnit.cpp') diff --git a/clang/lib/Frontend/ASTUnit.cpp b/clang/lib/Frontend/ASTUnit.cpp index 238ac339313..1f34f10f55a 100644 --- a/clang/lib/Frontend/ASTUnit.cpp +++ b/clang/lib/Frontend/ASTUnit.cpp @@ -924,9 +924,6 @@ public: class ASTUnitPreambleCallbacks : public PreambleCallbacks { public: - ASTUnitPreambleCallbacks(llvm::SmallVectorImpl &StoredDiags) - : StoredDiags(StoredDiags) {} - unsigned getHash() const { return Hash; } std::vector takeTopLevelDecls() { return std::move(TopLevelDecls); } @@ -964,7 +961,6 @@ public: } private: - llvm::SmallVectorImpl &StoredDiags; unsigned Hash = 0; std::vector TopLevelDecls; std::vector TopLevelDeclIDs; @@ -1259,7 +1255,7 @@ ASTUnit::getMainBufferWithPrecompiledPreamble( SmallVector NewPreambleDiagsStandalone; SmallVector NewPreambleDiags; - ASTUnitPreambleCallbacks Callbacks(NewPreambleDiags); + ASTUnitPreambleCallbacks Callbacks; { llvm::Optional Capture; if (CaptureDiagnostics) @@ -1282,18 +1278,16 @@ ASTUnit::getMainBufferWithPrecompiledPreamble( case BuildPreambleError::PreambleIsEmpty: // Try again next time. PreambleRebuildCounter = 1; - break; + return nullptr; case BuildPreambleError::CouldntCreateTargetInfo: case BuildPreambleError::BeginSourceFileFailed: case BuildPreambleError::CouldntEmitPCH: case BuildPreambleError::CouldntCreateVFSOverlay: // These erros are more likely to repeat, retry after some period. PreambleRebuildCounter = DefaultPreambleRebuildInterval; - break; - default: - llvm_unreachable("unexpected BuildPreambleError"); + return nullptr; } - return nullptr; + llvm_unreachable("unexpected BuildPreambleError"); } } -- cgit v1.2.3