[PATCH] D46603: [Support] Print TimeRecord as CSV

2018-05-08 Thread Roman Lebedev via Phabricator via cfe-commits
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 >

[PATCH] D46603: [Support] Print TimeRecord as CSV

2018-05-08 Thread George Karpenkov via Phabricator via cfe-commits
george.karpenkov added a comment. @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 Repository: rL LLVM

[PATCH] D46603: [Support] Print TimeRecord as CSV

2018-05-08 Thread Roman Lebedev via Phabricator via cfe-commits
lebedev.ri created this revision. lebedev.ri added reviewers: alexfh, sbenza, bkramer, george.karpenkov. This is needed for the continuation of https://reviews.llvm.org/D46504, to be able to store the timings as CSV. The floating-point values are dumped with no precision loss. See dependent