Merge pull request #1993 from compnerd/coverage

compnerd · web-flow · commit 46170cc2f5a9 · 2020-10-26T15:02:48.000-07:00
TypeSystem: silence some errors identified by MSVC
diff --git a/lldb/source/Plugins/TypeSystem/Swift/SwiftASTContext.cpp b/lldb/source/Plugins/TypeSystem/Swift/SwiftASTContext.cpp
@@ -1265,6 +1265,7 @@ static const char *getImportFailureString(swift::serialization::Status status) {
     return "The module file was built for a target newer than the current "
            "target.";
   }
+  llvm_unreachable("covered switch");
 }
 
 /// Initialize the compiler invocation with it the search paths from a
@@ -2991,6 +2992,7 @@ class SwiftDWARFImporterDelegate : public swift::DWARFImporterDelegate {
       // described in DWARF.
       return true;
     }
+    llvm_unreachable("covered switch");
   }
 
   clang::Decl *GetDeclForTypeAndKind(clang::QualType qual_type,

Original file line number	Diff line number	Diff line change
`@@ -1265,6 +1265,7 @@ static const char *getImportFailureString(swift::serialization::Status status) {`
`1265`	`1265`	`return "The module file was built for a target newer than the current "`
`1266`	`1266`	`"target.";`
`1267`	`1267`	`}`
	`1268`	`+ llvm_unreachable("covered switch");`
`1268`	`1269`	`}`
`1269`	`1270`
`1270`	`1271`	`/// Initialize the compiler invocation with it the search paths from a`
`@@ -2991,6 +2992,7 @@ class SwiftDWARFImporterDelegate : public swift::DWARFImporterDelegate {`
`2991`	`2992`	`// described in DWARF.`
`2992`	`2993`	`return true;`
`2993`	`2994`	`}`
	`2995`	`+ llvm_unreachable("covered switch");`
`2994`	`2996`	`}`
`2995`	`2997`
`2996`	`2998`	`clang::Decl *GetDeclForTypeAndKind(clang::QualType qual_type,`