[PATCH] D50439: [Tooling] Switch JSONCompilationDatabase to use JSON parser.

2018-08-08 Thread Sam McCall via Phabricator via cfe-commits
sammccall planned changes to this revision. sammccall added a comment. Nevermind, this is like 5x slower to load the chromium CDB. probably because of all the little non-arena allocations. I'll profile and fix it when I get bored again. Repository: rC Clang https://reviews.llvm.org/D50439

[PATCH] D50439: [Tooling] Switch JSONCompilationDatabase to use JSON parser.

2018-08-08 Thread Sam McCall via Phabricator via cfe-commits
sammccall updated this revision to Diff 159690. sammccall added a comment. (just updating description) Repository: rC Clang https://reviews.llvm.org/D50439 Files: include/clang/Tooling/JSONCompilationDatabase.h lib/Tooling/JSONCompilationDatabase.cpp test/Index/skip-parsed-bodies/compi

[PATCH] D50439: [Tooling] Switch JSONCompilationDatabase to use JSON parser.

2018-08-08 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. sammccall added reviewers: ioeric, klimek. Herald added a subscriber: cfe-commits. Rather than hold all the JSON source in memory, as well as the YAML structures needed to index into it, we parse eagerly into CompileCommand structures, which simplifies the implemen