================
@@ -139,16 +283,17 @@ FrontendAction::CreateWrappedASTConsumer(CompilerInstance 
&CI,
     return nullptr;
 
   // If there are no registered plugins we don't need to wrap the consumer
-  if (FrontendPluginRegistry::begin() == FrontendPluginRegistry::end())
-    return Consumer;
+  if (FrontendPluginRegistry::begin() == FrontendPluginRegistry::end()) {
----------------
VitaNuo wrote:

But what happens then if `FrontendPluginRegistry` isn't empty? We would skip 
extra plugins, which seems wrong.

https://github.com/llvm/llvm-project/pull/133910
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to