Skip to content

Commit

Permalink
[clang][Interp][NFC] Remove unneeded Source.h includes
Browse files Browse the repository at this point in the history
  • Loading branch information
tbaederr committed Sep 29, 2023
1 parent b242820 commit cc69662
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions clang/lib/AST/Interp/Source.h
Original file line number Diff line number Diff line change
Expand Up @@ -14,11 +14,15 @@
#define LLVM_CLANG_AST_INTERP_SOURCE_H

#include "PrimType.h"
#include "clang/AST/Decl.h"

This comment has been minimized.

Copy link
@sam-mccall

sam-mccall Oct 2, 2023

Collaborator

full definitions of Decl and Stmt are in fact required: PointerUnion uses alignof(T) to check that enough free bits are present at the bottom of T*.

So it seems this only compiles because all the TUs that use this in fact do include the full headers.
I've re-added the #includes in 6a01da4.

#include "clang/AST/Stmt.h"
#include "llvm/ADT/PointerUnion.h"
#include "llvm/Support/Endian.h"

namespace clang {
class Stmt;
class Decl;
class Expr;
class SourceLocation;
class SourceRange;
namespace interp {
class Function;

Expand Down

0 comments on commit cc69662

Please sign in to comment.