lebedev.ri added a comment.

In https://reviews.llvm.org/D46603#1092135, @george.karpenkov wrote:

> @lebedev.ri LLVM already has facilities for outputting statistics in JSON, it 
> seems that would be sufficient for your purposes?
>  I did something similar for the static analyzer in 
> https://reviews.llvm.org/D43131


YAML != JSON.
I did see all that `TimerGroup` stuff.
I will take a second look, but the `dumpVal()` change will still be needed 
either way.


Repository:
  rL LLVM

https://reviews.llvm.org/D46603



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to