[PATCH] D89277: [clangd] Add $/dumpMemoryTree LSP extension
Sam McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Oct 16 08:27:45 PDT 2020
sammccall accepted this revision.
sammccall added inline comments.
This revision is now accepted and ready to land.
================
Comment at: clang-tools-extra/clangd/ClangdLSPServer.cpp:623
{"typeHierarchyProvider", true},
+ {"memoryTreeProvider", true}, // clangd extension.
}}}};
----------------
one last naming nit: I think `memoryUsage` would put appropriately more weight on the semantics rather than the data structure
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D89277/new/
https://reviews.llvm.org/D89277
More information about the cfe-commits
mailing list