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

[Graydon Hoare via Phabricator via llvm-commits]

graydon added a comment.

In https://reviews.llvm.org/D31566#716897, @MatzeB wrote:

> Anyway without even the new system in place I cannot say much about how to transition to it... It'll surely be usably by code outside of LLVM, I guess we have to discuss how to transition users of the old system when we we know how the new one looks like.


Ok, cool! I'll keep my eyes open for a change; if you happen to decide / know a change is coming I'd appreciate a CC on it, but otherwise I'll just wait for the build to break :)

> Sounds like we have very similar problems. Unfortunately my attempts at making them cheaper failed with the current system because people rejected global constructors/variables (which however is the only way in the current system), see also http://lists.llvm.org/pipermail/llvm-dev/2016-December/108088.html

Yeah. It's tricky to balance all the concerns around globals/statics/inits. I'm fine with whatever happens, including status quo. It's usable enough.

Thanks!


https://reviews.llvm.org/D31566