rnk created this revision. rnk added a reviewer: rdhindsa. Herald added a project: clang. rdhindsa accepted this revision. rdhindsa added a comment. This revision is now accepted and ready to land.
LGTM After 60573ae6fe50 <https://reviews.llvm.org/rG60573ae6fe509b618dc6a2c5c55d921bccd77608> removed an include of Expr.h from ASTContext.h, this header fails to compile in some modular build configurations. I have not been able to reproduce the problem locally. The header compiles fine in isolation. However, based on reading the code, it seems like it would require Stmt to be complete. Based on that intuition, we decided to add the include. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D71222 Files: clang/include/clang/Tooling/Refactoring/ASTSelection.h Index: clang/include/clang/Tooling/Refactoring/ASTSelection.h =================================================================== --- clang/include/clang/Tooling/Refactoring/ASTSelection.h +++ clang/include/clang/Tooling/Refactoring/ASTSelection.h @@ -10,6 +10,7 @@ #define LLVM_CLANG_TOOLING_REFACTOR_AST_SELECTION_H #include "clang/AST/ASTTypeTraits.h" +#include "clang/AST/Stmt.h" #include "clang/Basic/LLVM.h" #include "clang/Basic/SourceLocation.h" #include <vector>
Index: clang/include/clang/Tooling/Refactoring/ASTSelection.h =================================================================== --- clang/include/clang/Tooling/Refactoring/ASTSelection.h +++ clang/include/clang/Tooling/Refactoring/ASTSelection.h @@ -10,6 +10,7 @@ #define LLVM_CLANG_TOOLING_REFACTOR_AST_SELECTION_H #include "clang/AST/ASTTypeTraits.h" +#include "clang/AST/Stmt.h" #include "clang/Basic/LLVM.h" #include "clang/Basic/SourceLocation.h" #include <vector>
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits