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

[Roman Tereshin via Phabricator via llvm-commits]

rtereshin added a comment.

Thanks! That's looking great!

As discussed, a separate patch could add a lock on that mutex in PrintStatistics and PrintStatisticsJSON to avoid the race.

I'm amazed that C++ standard library doesn't appear to have a Writer/Reader Lock, only the regular mutex. That's a shame. That thing could be useful here.

LGTM (again =))


Repository:
  rL LLVM

https://reviews.llvm.org/D43901