[PATCH] D52872: [clangd] Make binary index format the default, remove dead flag.

2018-10-05 Thread Sam McCall via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL343841: [clangd] Make binary index format the default, remove dead flag. (authored by sammccall, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.or

[PATCH] D52872: [clangd] Make binary index format the default, remove dead flag.

2018-10-05 Thread Haojian Wu via Phabricator via cfe-commits
hokein accepted this revision. hokein added a comment. This revision is now accepted and ready to land. looks good. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D52872 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://

[PATCH] D52872: [clangd] Make binary index format the default, remove dead flag.

2018-10-04 Thread Sam McCall via Phabricator via cfe-commits
sammccall marked an inline comment as done. sammccall added inline comments. Comment at: clangd/indexer/IndexerMain.cpp:58 SymbolCollector::Options Opts; -Opts.FallbackDir = AssumedHeaderDir; return createStaticIndexingAction( hokein wrote: > If we

[PATCH] D52872: [clangd] Make binary index format the default, remove dead flag.

2018-10-04 Thread Sam McCall via Phabricator via cfe-commits
sammccall updated this revision to Diff 168287. sammccall added a comment. example filename "clangd.dex" Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D52872 Files: clangd/indexer/IndexerMain.cpp Index: clangd/indexer/IndexerMain.cpp

[PATCH] D52872: [clangd] Make binary index format the default, remove dead flag.

2018-10-04 Thread Sam McCall via Phabricator via cfe-commits
sammccall added inline comments. Comment at: clangd/indexer/IndexerMain.cpp:84 - $ clangd-indexer --executor=all-TUs compile_commands.json > index.yaml + $ clangd-indexer --executor=all-TUs compile_commands.json > clangd-index hokein wrote: > ilya-biryukov

[PATCH] D52872: [clangd] Make binary index format the default, remove dead flag.

2018-10-04 Thread Haojian Wu via Phabricator via cfe-commits
hokein added inline comments. Comment at: clangd/indexer/IndexerMain.cpp:58 SymbolCollector::Options Opts; -Opts.FallbackDir = AssumedHeaderDir; return createStaticIndexingAction( If we remove the `assume-header-dir`, we probably remove `FallbackDir

[PATCH] D52872: [clangd] Make binary index format the default, remove dead flag.

2018-10-04 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov added inline comments. Comment at: clangd/indexer/IndexerMain.cpp:84 - $ clangd-indexer --executor=all-TUs compile_commands.json > index.yaml + $ clangd-indexer --executor=all-TUs compile_commands.json > clangd-index Maybe we should suggest a

[PATCH] D52872: [clangd] Make binary index format the default, remove dead flag.

2018-10-04 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. sammccall added a reviewer: hokein. Herald added subscribers: cfe-commits, kadircet, arphaman, jkorous, MaskRay, ioeric, ilya-biryukov. Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D52872 Files: clangd/indexer/IndexerMain.cpp Index: clangd/i