Re: [Lldb-commits] [PATCH] D20236: Remove ASTContexts from SystemInitializerCommon

2016-05-13 Thread Pavel Labath via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL269416: Remove ASTContexts from SystemInitializerCommon (authored by labath). Changed prior to commit: http://reviews.llvm.org/D20236?vs=57171&id=57175#toc Repository: rL LLVM http://reviews.llvm.or

Re: [Lldb-commits] [PATCH] D20236: Remove ASTContexts from SystemInitializerCommon

2016-05-13 Thread Tamas Berghammer via lldb-commits
tberghammer accepted this revision. tberghammer added a comment. This revision is now accepted and ready to land. Very nice! I don't know how we missed this optimization before. Finally the size of lldb-server starts to became more reasonable (3.7 MB on arm32) http://reviews.llvm.org/D20236

[Lldb-commits] [PATCH] D20236: Remove ASTContexts from SystemInitializerCommon

2016-05-13 Thread Pavel Labath via lldb-commits
labath created this revision. labath added reviewers: tberghammer, clayborg. labath added a subscriber: lldb-commits. The AST contexts are not needed in the server components, and the clang context in particular pulls in large parts of clang into the binary. Simply removing these two calls reduc