[PATCH] D43901: Make STATISTIC() values available programmatically

Roman Tereshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 1 22:43:51 PST 2018


rtereshin added a comment.

Ah, yeah, also, as the range returned is a couple of **vector** iterators, it looks like bumping any counter anywhere would potentially invalidate all the ranges existing. I suspect that might be a little error-prone and fragile even in a single-threaded environment. Maybe replace that vector with a list? And add a test that will check we don't regress on this little detail.


Repository:
  rL LLVM

https://reviews.llvm.org/D43901





More information about the llvm-commits mailing list