Index: include/clang/Sema/DeclSpec.h =================================================================== --- include/clang/Sema/DeclSpec.h (revision 140432) +++ include/clang/Sema/DeclSpec.h (working copy) @@ -1226,7 +1226,6 @@ void destroy() { switch (Kind) { - default: llvm_unreachable("Unknown decl type!"); case DeclaratorChunk::Function: return Fun.destroy(); case DeclaratorChunk::Pointer: return Ptr.destroy(); case DeclaratorChunk::BlockPointer: return Cls.destroy(); Index: include/clang/AST/RecursiveASTVisitor.h =================================================================== --- include/clang/AST/RecursiveASTVisitor.h (revision 140432) +++ include/clang/AST/RecursiveASTVisitor.h (working copy) @@ -1321,8 +1321,6 @@ case TSK_Undeclared: // Declaration of the template definition. case TSK_ExplicitSpecialization: break; - default: - llvm_unreachable("Unknown specialization kind."); } } Index: include/clang/AST/TypeVisitor.h =================================================================== --- include/clang/AST/TypeVisitor.h (revision 140432) +++ include/clang/AST/TypeVisitor.h (working copy) @@ -28,7 +28,6 @@ RetTy Visit(const Type *T) { // Top switch stmt: dispatch to VisitFooType for each FooType. switch (T->getTypeClass()) { - default: llvm_unreachable("Unknown type class!"); #define ABSTRACT_TYPE(CLASS, PARENT) #define TYPE(CLASS, PARENT) case Type::CLASS: DISPATCH(CLASS##Type); #include "clang/AST/TypeNodes.def" Index: include/clang/AST/StmtVisitor.h =================================================================== --- include/clang/AST/StmtVisitor.h (revision 140432) +++ include/clang/AST/StmtVisitor.h (working copy) @@ -42,7 +42,6 @@ // below. if (PTR(BinaryOperator) BinOp = dyn_cast(S)) { switch (BinOp->getOpcode()) { - default: llvm_unreachable("Unknown binary operator!"); case BO_PtrMemD: DISPATCH(BinPtrMemD, BinaryOperator); case BO_PtrMemI: DISPATCH(BinPtrMemI, BinaryOperator); case BO_Mul: DISPATCH(BinMul, BinaryOperator); @@ -80,7 +79,6 @@ } } else if (PTR(UnaryOperator) UnOp = dyn_cast(S)) { switch (UnOp->getOpcode()) { - default: llvm_unreachable("Unknown unary operator!"); case UO_PostInc: DISPATCH(UnaryPostInc, UnaryOperator); case UO_PostDec: DISPATCH(UnaryPostDec, UnaryOperator); case UO_PreInc: DISPATCH(UnaryPreInc, UnaryOperator); Index: include/clang/AST/DeclVisitor.h =================================================================== --- include/clang/AST/DeclVisitor.h (revision 140432) +++ include/clang/AST/DeclVisitor.h (working copy) @@ -30,7 +30,6 @@ public: RetTy Visit(Decl *D) { switch (D->getKind()) { - default: llvm_unreachable("Decl that isn't part of DeclNodes.inc!"); #define DECL(DERIVED, BASE) \ case Decl::DERIVED: DISPATCH(DERIVED##Decl, DERIVED##Decl); #define ABSTRACT_DECL(DECL) Index: lib/Frontend/LangStandards.cpp =================================================================== --- lib/Frontend/LangStandards.cpp (revision 140432) +++ lib/Frontend/LangStandards.cpp (working copy) @@ -19,8 +19,6 @@ const LangStandard &LangStandard::getLangStandardForKind(Kind K) { switch (K) { - default: - llvm_unreachable("Invalid language kind!"); case lang_unspecified: llvm::report_fatal_error("getLangStandardForKind() on unspecified kind"); #define LANGSTANDARD(id, name, desc, features) \ Index: lib/Frontend/LogDiagnosticPrinter.cpp =================================================================== --- lib/Frontend/LogDiagnosticPrinter.cpp (revision 140432) +++ lib/Frontend/LogDiagnosticPrinter.cpp (working copy) @@ -28,8 +28,6 @@ static StringRef getLevelName(Diagnostic::Level Level) { switch (Level) { - default: - return ""; case Diagnostic::Ignored: return "ignored"; case Diagnostic::Note: return "note"; case Diagnostic::Warning: return "warning"; Index: lib/Sema/SemaDeclAttr.cpp =================================================================== --- lib/Sema/SemaDeclAttr.cpp (revision 140432) +++ lib/Sema/SemaDeclAttr.cpp (working copy) @@ -1049,8 +1049,6 @@ } break; } - default: - llvm_unreachable("Unknown ownership attribute"); } // switch // Check we don't have a conflict with another ownership attribute. Index: lib/Sema/SemaInit.cpp =================================================================== --- lib/Sema/SemaInit.cpp (revision 140432) +++ lib/Sema/SemaInit.cpp (working copy) @@ -2461,7 +2461,6 @@ case VK_RValue: S.Kind = SK_CastDerivedToBaseRValue; break; case VK_XValue: S.Kind = SK_CastDerivedToBaseXValue; break; case VK_LValue: S.Kind = SK_CastDerivedToBaseLValue; break; - default: llvm_unreachable("No such category"); } S.Type = BaseType; Steps.push_back(S); Index: lib/Sema/SemaDecl.cpp =================================================================== --- lib/Sema/SemaDecl.cpp (revision 140432) +++ lib/Sema/SemaDecl.cpp (working copy) @@ -245,7 +245,6 @@ if (R.getResultKind() == LookupResult::Found) if (const TagDecl *TD = R.getAsSingle()) { switch (TD->getTagKind()) { - default: return DeclSpec::TST_unspecified; case TTK_Struct: return DeclSpec::TST_struct; case TTK_Union: return DeclSpec::TST_union; case TTK_Class: return DeclSpec::TST_class; Index: lib/Sema/SemaType.cpp =================================================================== --- lib/Sema/SemaType.cpp (revision 140432) +++ lib/Sema/SemaType.cpp (working copy) @@ -1921,7 +1921,6 @@ state.setCurrentChunkIndex(chunkIndex); DeclaratorChunk &DeclType = D.getTypeObject(chunkIndex); switch (DeclType.Kind) { - default: llvm_unreachable("Unknown decltype!"); case DeclaratorChunk::Paren: T = S.BuildParenType(T); break; Index: lib/Sema/SemaAttr.cpp =================================================================== --- lib/Sema/SemaAttr.cpp (revision 140432) +++ lib/Sema/SemaAttr.cpp (working copy) @@ -263,9 +263,6 @@ Context->setAlignment(AlignmentVal); } break; - - default: - llvm_unreachable("Invalid #pragma pack kind."); } } Index: lib/Sema/SemaExpr.cpp =================================================================== --- lib/Sema/SemaExpr.cpp (revision 140432) +++ lib/Sema/SemaExpr.cpp (working copy) @@ -8992,7 +8992,6 @@ bool MayHaveConvFixit = false; switch (ConvTy) { - default: llvm_unreachable("Unknown conversion type"); case Compatible: return false; case PointerToInt: DiagKind = diag::ext_typecheck_convert_pointer_int; Index: lib/AST/TypePrinter.cpp =================================================================== --- lib/AST/TypePrinter.cpp (revision 140432) +++ lib/AST/TypePrinter.cpp (working copy) @@ -426,8 +426,7 @@ FunctionType::ExtInfo Info = T->getExtInfo(); switch(Info.getCC()) { - case CC_Default: - default: break; + case CC_Default: break; case CC_C: S += " __attribute__((cdecl))"; break; Index: lib/AST/DeclBase.cpp =================================================================== --- lib/AST/DeclBase.cpp (revision 140432) +++ lib/AST/DeclBase.cpp (working copy) @@ -100,7 +100,6 @@ void Decl::add(Kind k) { switch (k) { - default: llvm_unreachable("Declaration not in DeclNodes.inc!"); #define DECL(DERIVED, BASE) case DERIVED: ++n##DERIVED##s; break; #define ABSTRACT_DECL(DECL) #include "clang/AST/DeclNodes.inc" Index: lib/AST/Expr.cpp =================================================================== --- lib/AST/Expr.cpp (revision 140432) +++ lib/AST/Expr.cpp (working copy) @@ -595,7 +595,6 @@ /// corresponds to, e.g. "sizeof" or "[pre]++". const char *UnaryOperator::getOpcodeStr(Opcode Op) { switch (Op) { - default: llvm_unreachable("Unknown unary operator"); case UO_PostInc: return "++"; case UO_PostDec: return "--"; case UO_PreInc: return "++"; Index: lib/AST/DumpXML.cpp =================================================================== --- lib/AST/DumpXML.cpp (revision 140432) +++ lib/AST/DumpXML.cpp (working copy) @@ -66,7 +66,6 @@ void dispatch(Decl *D) { switch (D->getKind()) { - default: llvm_unreachable("Decl that isn't part of DeclNodes.inc!"); #define DECL(DERIVED, BASE) \ case Decl::DERIVED: \ DISPATCH(dispatch##DERIVED##DeclAttrs, DERIVED##Decl); \ @@ -121,7 +120,6 @@ void dispatch(Type *T) { switch (T->getTypeClass()) { - default: llvm_unreachable("Type that isn't part of TypeNodes.inc!"); #define TYPE(DERIVED, BASE) \ case Type::DERIVED: \ DISPATCH(dispatch##DERIVED##TypeAttrs, DERIVED##Type); \ Index: lib/AST/DeclCXX.cpp =================================================================== --- lib/AST/DeclCXX.cpp (revision 140432) +++ lib/AST/DeclCXX.cpp (working copy) @@ -1691,7 +1691,6 @@ static const char *getAccessName(AccessSpecifier AS) { switch (AS) { - default: case AS_none: llvm_unreachable("Invalid access specifier!"); case AS_public: Index: lib/AST/ASTContext.cpp =================================================================== --- lib/AST/ASTContext.cpp (revision 140432) +++ lib/AST/ASTContext.cpp (working copy) @@ -3447,7 +3447,6 @@ FloatingRank EltRank = getFloatingRank(Size); if (Domain->isComplexType()) { switch (EltRank) { - default: llvm_unreachable("getFloatingRank(): illegal value for rank"); case FloatRank: return FloatComplexTy; case DoubleRank: return DoubleComplexTy; case LongDoubleRank: return LongDoubleComplexTy; @@ -3456,7 +3455,6 @@ assert(Domain->isRealFloatingType() && "Unknown domain!"); switch (EltRank) { - default: llvm_unreachable("getFloatingRank(): illegal value for rank"); case FloatRank: return FloatTy; case DoubleRank: return DoubleTy; case LongDoubleRank: return LongDoubleTy; Index: lib/AST/APValue.cpp =================================================================== --- lib/AST/APValue.cpp (revision 140432) +++ lib/AST/APValue.cpp (working copy) @@ -94,7 +94,6 @@ void APValue::print(raw_ostream &OS) const { switch (getKind()) { - default: llvm_unreachable("Unknown APValue kind!"); case Uninitialized: OS << "Uninitialized"; return; @@ -124,7 +123,6 @@ static void WriteShortAPValueToStream(raw_ostream& Out, const APValue& V) { switch (V.getKind()) { - default: llvm_unreachable("Unknown APValue kind!"); case APValue::Uninitialized: Out << "Uninitialized"; break; Index: lib/Driver/Option.cpp =================================================================== --- lib/Driver/Option.cpp (revision 140432) +++ lib/Driver/Option.cpp (working copy) @@ -61,8 +61,6 @@ void Option::dump() const { llvm::errs() << "<"; switch (Kind) { - default: - llvm_unreachable("Invalid kind"); #define P(N) case N: llvm::errs() << #N; break P(GroupClass); P(InputClass); Index: lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp =================================================================== --- lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp (revision 140432) +++ lib/StaticAnalyzer/Frontend/AnalysisConsumer.cpp (working copy) @@ -337,8 +337,6 @@ Decl *D) { switch (mgr.getLangOptions().getGC()) { - default: - llvm_unreachable("Invalid GC mode."); case LangOptions::NonGC: ActionExprEngine(C, mgr, D, false); break; Index: lib/StaticAnalyzer/Checkers/DeadStoresChecker.cpp =================================================================== --- lib/StaticAnalyzer/Checkers/DeadStoresChecker.cpp (revision 140432) +++ lib/StaticAnalyzer/Checkers/DeadStoresChecker.cpp (working copy) @@ -109,9 +109,6 @@ const char *BugType = 0; switch (dsk) { - default: - llvm_unreachable("Impossible dead store type."); - case DeadInit: BugType = "Dead initialization"; os << "Value stored to '" << V Index: lib/StaticAnalyzer/Checkers/RetainCountChecker.cpp =================================================================== --- lib/StaticAnalyzer/Checkers/RetainCountChecker.cpp (revision 140432) +++ lib/StaticAnalyzer/Checkers/RetainCountChecker.cpp (working copy) @@ -1097,7 +1097,6 @@ case cfretain: Effect = IncRef; break; case cfrelease: Effect = DecRef; break; case cfmakecollectable: Effect = MakeCollectable; break; - default: llvm_unreachable("Not a supported unary function."); } ScratchArgs = AF.add(ScratchArgs, 0, Effect); Index: lib/StaticAnalyzer/Checkers/MallocChecker.cpp =================================================================== --- lib/StaticAnalyzer/Checkers/MallocChecker.cpp (revision 140432) +++ lib/StaticAnalyzer/Checkers/MallocChecker.cpp (working copy) @@ -184,8 +184,6 @@ rv = true; break; } - default: - break; } } } Index: lib/StaticAnalyzer/Core/SVals.cpp =================================================================== --- lib/StaticAnalyzer/Core/SVals.cpp (revision 140432) +++ lib/StaticAnalyzer/Core/SVals.cpp (working copy) @@ -281,8 +281,6 @@ case UndefinedKind: os << "Undefined"; break; - default: - assert (false && "Invalid SVal."); } } Index: lib/CodeGen/CGObjCMac.cpp =================================================================== --- lib/CodeGen/CGObjCMac.cpp (revision 140432) +++ lib/CodeGen/CGObjCMac.cpp (working copy) @@ -4633,8 +4633,6 @@ // At various points we've experimented with using vtable-based // dispatch for all methods. switch (CGM.getCodeGenOpts().getObjCDispatchMethod()) { - default: - llvm_unreachable("Invalid dispatch method!"); case CodeGenOptions::Legacy: return false; case CodeGenOptions::NonLegacy: Index: lib/Index/ASTLocation.cpp =================================================================== --- lib/Index/ASTLocation.cpp (revision 140432) +++ lib/Index/ASTLocation.cpp (working copy) @@ -41,7 +41,6 @@ return 0; switch (getKind()) { - default: llvm_unreachable("Invalid Kind"); case N_Type: return 0; case N_Decl: @@ -60,7 +59,6 @@ return SourceRange(); switch (getKind()) { - default: llvm_unreachable("Invalid Kind"); case N_Decl: return D->getSourceRange(); case N_Stmt: Index: lib/Parse/ParseExprCXX.cpp =================================================================== --- lib/Parse/ParseExprCXX.cpp (revision 140432) +++ lib/Parse/ParseExprCXX.cpp (working copy) @@ -2375,8 +2375,6 @@ return Actions.ActOnArrayTypeTrait(ATT, Loc, Ty.get(), DimExpr.get(), RParen); } - default: - break; } return ExprError(); } Index: lib/Serialization/ASTReaderDecl.cpp =================================================================== --- lib/Serialization/ASTReaderDecl.cpp (revision 140432) +++ lib/Serialization/ASTReaderDecl.cpp (working copy) @@ -345,7 +345,6 @@ ReadDeclarationNameLoc(FD->DNLoc, FD->getDeclName(), Record, Idx); FD->IdentifierNamespace = Record[Idx++]; switch ((FunctionDecl::TemplatedKind)Record[Idx++]) { - default: llvm_unreachable("Unhandled TemplatedKind!"); case FunctionDecl::TK_NonTemplate: break; case FunctionDecl::TK_FunctionTemplate: @@ -953,8 +952,6 @@ CXXRecNotTemplate = 0, CXXRecTemplate, CXXRecMemberSpecialization }; switch ((CXXRecKind)Record[Idx++]) { - default: - llvm_unreachable("Out of sync with ASTDeclWriter::VisitCXXRecordDecl?"); case CXXRecNotTemplate: break; case CXXRecTemplate: @@ -1304,9 +1301,6 @@ enum RedeclKind { NoRedeclaration = 0, PointsToPrevious, PointsToLatest }; RedeclKind Kind = (RedeclKind)Record[Idx++]; switch (Kind) { - default: - llvm_unreachable("Out of sync with ASTDeclWriter::VisitRedeclarable or" - " messed up reading"); case NoRedeclaration: break; case PointsToPrevious: { Index: lib/Serialization/ASTWriterDecl.cpp =================================================================== --- lib/Serialization/ASTWriterDecl.cpp (revision 140432) +++ lib/Serialization/ASTWriterDecl.cpp (working copy) @@ -295,7 +295,6 @@ Record.push_back(D->getIdentifierNamespace()); Record.push_back(D->getTemplatedKind()); switch (D->getTemplatedKind()) { - default: llvm_unreachable("Unhandled TemplatedKind!"); case FunctionDecl::TK_NonTemplate: break; case FunctionDecl::TK_FunctionTemplate: Index: lib/FrontendTool/ExecuteCompilerInvocation.cpp =================================================================== --- lib/FrontendTool/ExecuteCompilerInvocation.cpp (revision 140432) +++ lib/FrontendTool/ExecuteCompilerInvocation.cpp (working copy) @@ -32,9 +32,6 @@ using namespace clang::frontend; switch (CI.getFrontendOpts().ProgramAction) { - default: - llvm_unreachable("Invalid program action!"); - case ASTDump: return new ASTDumpAction(); case ASTDumpXML: return new ASTDumpXMLAction(); case ASTPrint: return new ASTPrintAction();