kbobyrev added inline comments.
================
Comment at: clang-tools-extra/clangd/benchmarks/IndexBenchmark.cpp:96
-static void DexQueries(benchmark::State &State) {
+// This is not a *real* benchmark: it shows size of built MemIndex (in bytes).
+// Same for the next "benchmark".
----------------
sammccall wrote:
> This looks like it'll be really stable, why does it need to be a benchmark vs
> say a dexp subcommand?
As discussed offline, this is meant to make it easier for people to investigate
memory + performance changes and simplify the development pipeline as opposed
to remembering multiple binaries and their options and running all these
binaries after each change.
https://reviews.llvm.org/D52047
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
