[Lldb-commits] [PATCH] D18530: Move some functions from ClangASTContext to ClangUtil

[Pavel Labath via lldb-commits]

labath added a subscriber: labath.
labath added a comment.

Cyclic code dependencies are not an imaginary problem. Right now, it is very hard to make lldb-server small, because everything in lldb depends on everything else, and so lldb-server ends up containing chunks e.g. clang, even though that code will never get execute, because it is impossible to remove it without causing link errors. So, I welcome anything which makes this task easier, now or in the future. Breaking things up into smaller classes/files is definitely one of those things. I also welcome more unittests.


http://reviews.llvm.org/D18530