[PATCH] D31566: [Support] Make printAllJSONValues public, for custom output.

[Matthias Braun via Phabricator via llvm-commits]

MatzeB accepted this revision.
MatzeB added a comment.
This revision is now accepted and ready to land.

We could go ahead and make this function public for now.

However I don't expect this API to be stable. Long term we really should move away from a global list of timers accessed by TimerGroup. In fact I am planning changes to the statistics system right now. Are you prepared to loose the function again in a few weeks?

Also the fact that you need to call this function seems suspicious to me. Could you elaborate a bit why PrintStatisticsJSON() wasn't enough for your use case?


https://reviews.llvm.org/D31566