r350453 - Let new test from r350340 still pass even after r350451.
Nico Weber via cfe-commits
cfe-commits at lists.llvm.org
Fri Jan 4 17:19:14 PST 2019
Author: nico
Date: Fri Jan 4 17:19:14 2019
New Revision: 350453
URL: http://llvm.org/viewvc/llvm-project?rev=350453&view=rev
Log:
Let new test from r350340 still pass even after r350451.
Modified:
cfe/trunk/lib/Frontend/FrontendAction.cpp
Modified: cfe/trunk/lib/Frontend/FrontendAction.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/FrontendAction.cpp?rev=350453&r1=350452&r2=350453&view=diff
==============================================================================
--- cfe/trunk/lib/Frontend/FrontendAction.cpp (original)
+++ cfe/trunk/lib/Frontend/FrontendAction.cpp Fri Jan 4 17:19:14 2019
@@ -152,10 +152,6 @@ FrontendAction::CreateWrappedASTConsumer
if (!Consumer)
return nullptr;
- // If there are no registered plugins we don't need to wrap the consumer
- if (FrontendPluginRegistry::begin() == FrontendPluginRegistry::end())
- return Consumer;
-
// Validate -add-plugin args.
bool FoundAllPlugins = true;
for (const std::string &Arg : CI.getFrontendOpts().AddPluginActions) {
@@ -174,6 +170,10 @@ FrontendAction::CreateWrappedASTConsumer
if (!FoundAllPlugins)
return nullptr;
+ // If there are no registered plugins we don't need to wrap the consumer
+ if (FrontendPluginRegistry::begin() == FrontendPluginRegistry::end())
+ return Consumer;
+
// If this is a code completion run, avoid invoking the plugin consumers
if (CI.hasCodeCompletionConsumer())
return Consumer;
More information about the cfe-commits
mailing list