[Lldb-commits] [PATCH] D71336: [lldb] Remove ClangASTMetrics

2019-12-12 Thread Raphael Isemann via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGe39cb48cd0bd: [lldb] Remove ClangASTMetrics (authored by teemperor). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71336/new/ https://reviews.llvm.org/D7133

[Lldb-commits] [PATCH] D71336: [lldb] Remove ClangASTMetrics

2019-12-11 Thread Shafik Yaghmour via Phabricator via lldb-commits
shafik accepted this revision. shafik added a comment. This revision is now accepted and ready to land. Herald added a subscriber: rnkovacs. It is not clear to me we could use these in a test because I don't see a reason for the numbers to stay stable and therefore a useful measure. Repository:

[Lldb-commits] [PATCH] D71336: [lldb] Remove ClangASTMetrics

2019-12-11 Thread Raphael Isemann via Phabricator via lldb-commits
teemperor created this revision. Herald added a reviewer: martong. Herald added a reviewer: shafik. Herald added subscribers: lldb-commits, JDevlieghere. Herald added a project: LLDB. Not once have I looked at these numbers in a log and considered them useful for anything. Also we should not impl