r215353 - Properly #include ASTConsumer.h instead of forward declaration.

Daniel Jasper djasper at google.com
Mon Aug 11 04:37:33 PDT 2014


Author: djasper
Date: Mon Aug 11 06:37:33 2014
New Revision: 215353

URL: http://llvm.org/viewvc/llvm-project?rev=215353&view=rev
Log:
Properly #include ASTConsumer.h instead of forward declaration.

Otherwise, this can lead to compile failures if a user/subclass of
FrontendAction doesn't #include this file, even if CreateASTConsumer
isn't used directly.

Modified:
    cfe/trunk/include/clang/Frontend/FrontendAction.h

Modified: cfe/trunk/include/clang/Frontend/FrontendAction.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Frontend/FrontendAction.h?rev=215353&r1=215352&r2=215353&view=diff
==============================================================================
--- cfe/trunk/include/clang/Frontend/FrontendAction.h (original)
+++ cfe/trunk/include/clang/Frontend/FrontendAction.h Mon Aug 11 06:37:33 2014
@@ -18,6 +18,7 @@
 #ifndef LLVM_CLANG_FRONTEND_FRONTENDACTION_H
 #define LLVM_CLANG_FRONTEND_FRONTENDACTION_H
 
+#include "clang/AST/ASTConsumer.h"
 #include "clang/Basic/LLVM.h"
 #include "clang/Basic/LangOptions.h"
 #include "clang/Frontend/ASTUnit.h"
@@ -28,7 +29,6 @@
 #include <vector>
 
 namespace clang {
-class ASTConsumer;
 class ASTMergeAction;
 class CompilerInstance;
 





More information about the cfe-commits mailing list