This is an archive of the discontinued LLVM Phabricator instance.

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".
This breaks the modules build.

Diff Detail

Repository
rL LLVM

Event Timeline

bader updated this revision to Diff 53213.Apr 11 2016, 4:07 AM
bader retitled this revision from to Move OpenCLImageTypes.def from clangAST to clangBasic library..
bader updated this object.
bader added a reviewer: rsmith.
rsmith accepted this revision.Apr 12 2016, 1:29 PM
rsmith edited edge metadata.
This revision is now accepted and ready to land.Apr 12 2016, 1:29 PM
This revision was automatically updated to reflect the committed changes.
cfe/trunk/include/clang/AST/ASTContext.h