[cfe-commits] Bug 11709 Fix: va_list on ARM is not following AAPCS 7.1.4

[Sean Silva]

> 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 <jordan_rose at apple.com> wrote:
>
> On Oct 4, 2012, at 18:21 , Sean Silva <silvas at purdue.edu> 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.
>