Skip to content

Commit

Permalink
Include Stmt.h where it seems to be necessary for modules builds
Browse files Browse the repository at this point in the history
Summary:
After 60573ae 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.

Reviewers: rdhindsa

Subscribers: cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D71222
  • Loading branch information
rnk committed Dec 9, 2019
1 parent d9ae493 commit 1164d43
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions clang/include/clang/Tooling/Refactoring/ASTSelection.h
Original file line number Diff line number Diff line change
Expand Up @@ -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>
Expand Down

0 comments on commit 1164d43

Please sign in to comment.