[PATCH] D51276: [New PM][PassTiming] implement -time-passes for the new pass manager

Thu Oct 4 03:38:14 PDT 2018

chandlerc added a comment.

In https://reviews.llvm.org/D51276#1254913, @fedor.sergeev wrote:

> as for the per-pass-name aggregation - I was going to supply a follow-up patch that changes -time-passes from boolean option to -time-passes=<aggregation-type>.
>  Perhaps aggregation-type should be 'aggregate' by default, and 'invocations' providing per-invocation info.
>  Yet I dont want to just remove the possibility of getting a more fine-grained data that we have in this patch.

Yeah, i definitely like having the per-invocation information. I'm just suggesting that (eventually, happy for this to come in follow-ups) we'll probably want the default to be aggregated a bit.

Repository:
  rL LLVM

https://reviews.llvm.org/D51276