[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