[PATCH] D52233: [dexp] Allow users to dump JSON representations of fuzzy find requests

2018-09-19 Thread Eric Liu via Phabricator via cfe-commits
ioeric added inline comments. Comment at: clang-tools-extra/clangd/index/dex/dexp/Dexp.cpp:131 } +if (ShowJSONRequest) + llvm::outs() << llvm::formatv("Request:\n{0}\n\n", toJSON(Request)); Any reason not to print this always? https://reviews.llvm

[PATCH] D52233: [dexp] Allow users to dump JSON representations of fuzzy find requests

2018-09-18 Thread Kirill Bobyrev via Phabricator via cfe-commits
kbobyrev created this revision. kbobyrev added reviewers: ioeric, ilya-biryukov. kbobyrev added a project: clang-tools-extra. Herald added subscribers: kadircet, arphaman, jkorous. This might be useful for benchmark construction. https://reviews.llvm.org/D52233 Files: clang-tools-extra/clangd