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

Kadir Cetinkaya via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 5 00:35:36 PST 2020


kadircet added a comment.

In D75603#1906418 <https://reviews.llvm.org/D75603#1906418>, @sammccall wrote:

> 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.


as discussed offline, I was rather afraid of the initial version of the patch, but the final version seems ok as it only adds a single field to codecompleteopts.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75603





More information about the cfe-commits mailing list