[PATCH] D51971: [clangd] Use JSON format in benchmark requests reader

[Kirill Bobyrev via Phabricator via cfe-commits]

kbobyrev added inline comments.


================
Comment at: clang-tools-extra/clangd/benchmarks/IndexBenchmark.cpp:45
+  if (!JSONArray) {
+    llvm::errs() << "Couldn't parse request.\n";
+  }
----------------
ilya-biryukov wrote:
> Return from function after error?
I thought that this should panic instead of returning empty `Requests`: otherwise it wouldn't be possible to detect problems in the benchmark driver in test, for example (it will just run benchmark over empty list of requests instead of doing something useful).


https://reviews.llvm.org/D51971