[PATCH] D58572: [Support] allow -stats/-time-passes reporting into a custom stream

Philip Pfaffe via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 25 04:10:49 PST 2019


philip.pfaffe added a comment.

I don't think this is sufficient or goes into the right direction to support parallel info collection and output.

For the new PassManager, reporting per-thread timings is trivial already and requires no (additional) global state. You can just create a TimePassesHandler instance per thread or per pipeline.
For Statistics on the other hand the approach in this change is insufficient, because the statistics registry is already a singleton. In order to support parallel statistics collection, can we get rid of the Statistics singleton?


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58572/new/

https://reviews.llvm.org/D58572





More information about the llvm-commits mailing list