[PATCH] D88590: [clangd] Add benchmark for measuring latency of DecisionForest model.

Adam Czachorowski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 2 07:01:23 PDT 2020


adamcz accepted this revision.
adamcz added a comment.
This revision is now accepted and ready to land.

Still a couple of lint warnings, otherwise LGTM



================
Comment at: clang-tools-extra/clangd/benchmarks/CompletionModel/DecisionForestBenchmark.cpp:74
+  for (auto _ : State) {
+    srand(0);
+    State.PauseTiming();
----------------
Move this out of the loop. Right now we re-generate "random" dataset, but it's always the same dataset ;-)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D88590/new/

https://reviews.llvm.org/D88590



More information about the cfe-commits mailing list