Putting OpenCLImageTypes.def to clangAST library violates layering requirement: "It's not OK for a Basic/ header to include an AST/ header".
This breaks the modules build.
Details
Details
Diff Detail
Diff Detail
Paths
| Differential D18954
Move OpenCLImageTypes.def from clangAST to clangBasic library. ClosedPublic Authored by bader on Apr 11 2016, 4:07 AM.
Details Summary Putting OpenCLImageTypes.def to clangAST library violates layering requirement: "It's not OK for a Basic/ header to include an AST/ header".
Diff Detail Event TimelineThis revision is now accepted and ready to land.Apr 12 2016, 1:29 PM Closed by commit rL266180: [OpenCL] Move OpenCLImageTypes.def from clangAST to clangBasic library. (authored by bader). · Explain WhyApr 13 2016, 1:39 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 53213 include/clang/AST/ASTContext.h
include/clang/AST/OpenCLImageTypes.def
include/clang/AST/Type.h
include/clang/Basic/OpenCLImageTypes.def
include/clang/Basic/Specifiers.h
include/clang/Basic/TokenKinds.def
include/clang/Sema/DeclSpec.h
include/clang/Serialization/ASTBitCodes.h
lib/AST/ASTContext.cpp
lib/AST/ASTImporter.cpp
lib/AST/ExprConstant.cpp
lib/AST/ItaniumMangle.cpp
lib/AST/MicrosoftMangle.cpp
lib/AST/NSAPI.cpp
lib/AST/Type.cpp
lib/AST/TypeLoc.cpp
lib/Analysis/PrintfFormatString.cpp
lib/CodeGen/CGDebugInfo.h
lib/CodeGen/CGDebugInfo.cpp
lib/CodeGen/CGOpenCLRuntime.cpp
lib/CodeGen/CodeGenTypes.cpp
lib/CodeGen/ItaniumCXXABI.cpp
lib/Index/USRGeneration.cpp
lib/Parse/ParseDecl.cpp
lib/Parse/ParseExpr.cpp
lib/Parse/ParseTentative.cpp
lib/Sema/DeclSpec.cpp
lib/Sema/SemaExpr.cpp
lib/Sema/SemaTemplateVariadic.cpp
lib/Sema/SemaType.cpp
lib/Serialization/ASTCommon.cpp
lib/Serialization/ASTReader.cpp
tools/libclang/CIndex.cpp
|