Fix missing return on error path in loadExecutablePlugin().

DougGregor · DougGregor · commit eb2fa4498e69 · 2023-04-18T18:12:22.000-07:00
diff --git a/lib/AST/ASTContext.cpp b/lib/AST/ASTContext.cpp
@@ -6393,6 +6393,7 @@ LoadedExecutablePlugin *ASTContext::loadExecutablePlugin(StringRef path) {
   if (!plugin) {
     Diags.diagnose(SourceLoc(), diag::compiler_plugin_not_loaded, path,
                    llvm::toString(plugin.takeError()));
+    return nullptr;
   }
 
   return plugin.get();

Original file line number	Diff line number	Diff line change
`@@ -6393,6 +6393,7 @@ LoadedExecutablePlugin *ASTContext::loadExecutablePlugin(StringRef path) {`
`6393`	`6393`	`if (!plugin) {`
`6394`	`6394`	`Diags.diagnose(SourceLoc(), diag::compiler_plugin_not_loaded, path,`
`6395`	`6395`	`llvm::toString(plugin.takeError()));`
	`6396`	`+ return nullptr;`
`6396`	`6397`	`}`
`6397`	`6398`
`6398`	`6399`	`return plugin.get();`