[PATCH] D75603: [clangd] Add instrumentation mode in clangd for metrics collection.

[Sam McCall via Phabricator via cfe-commits]

sammccall added a comment.

Forgot to mention: I also think the trace approach certainly has things going for it, or even parsing out the messages from the existing logs.
But in this particular case the callback happens to be extremely convenient and also not invasive (since the data structures are already exposed, code complete has an opts struct etc). And since this is for analysis we have a lot of flexibility to rework if it stops being easy to maintain.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75603