[PATCH] D48109: [Timers] Use the pass argument name for JSON keys in time-passes

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 13 11:33:21 PDT 2018


MatzeB accepted this revision.
MatzeB added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D48109#1131367, @thegameg wrote:

> @chandlerc @jlebar can you confirm that the new PM doesn't have timer support?


I'm pretty sure I only added the timer->statistics support for the "legacy" pass manager.

LGTM.

I suspect there aren't too many people tracking those stats yet so renaming should be fine.

(I also wish pass names and argument names would just be the same, but this is fine for the status quo).


https://reviews.llvm.org/D48109





More information about the llvm-commits mailing list