> Sema depends on AST, and we don't allow circular dependencies. I guess it > would be nice if we had a map of all our dependencies somewhere; bonus points > if it's autogenerated. Double bonus points if the SVN server rejects commits > that cause a circular dependency.
I guess the "would be nice" part if fulfilled by page 3 of <http://wiki.ifs.hsr.ch/SemProgAnTr/files/Clang_Architecture_-_ChristopherGuntli.pdf>.. It looks like Doxygen also has some information about dependencies on its "directory" pages. For example, there appears to be an alarming amount of circularity on <http://clang.llvm.org/doxygen/dir_36b800a3fe94cf7aedd3066cf4f06787.html> (primarily, it seems, related to libBasic). It looks like most are just strays, e.g. with 31 dependencies in one direction and 1 in the other. -- Sean Silva On Thu, Oct 4, 2012 at 9:23 PM, Jordan Rose <[email protected]> wrote: > > On Oct 4, 2012, at 18:21 , Sean Silva <[email protected]> wrote: > >>> - AST can't depend on Sema. Is there a reason for the include of Sema.h in >>> ASTContext.cpp? >> >> Is this documented anywhere? If not you should mention it somewhere. > > Sema depends on AST, and we don't allow circular dependencies. I guess it > would be nice if we had a map of all our dependencies somewhere; bonus points > if it's autogenerated. Double bonus points if the SVN server rejects commits > that cause a circular dependency. > _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
