[PATCH] D88417: [clangd] Record memory usages after each notification

Sam McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 7 09:54:30 PDT 2020


sammccall added a comment.

this is at least going to take some important locks, hopefully only briefly.

We should watch the timing here carefully and consider guarding it - apart from the minimum time interval we discussed, we could have a check whether metric tracing is actually enabled in a meaningful way.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88417



More information about the cfe-commits mailing list