[PATCH] D52356: [PassTiming] cleaning up legacy PassTimingInfo interface. NFCI.

Fedor Sergeev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 24 07:21:28 PDT 2018


fedor.sergeev added a comment.

In https://reviews.llvm.org/D52356#1243219, @philip.pfaffe wrote:

> is this in some sense a revert of https://reviews.llvm.org/rL340872? Which parts of that change remain?


It is not a complete revert, but definitely very close to it.
One of the changes that were not restored to the original form is removal of createTheTimeInfo, which had been replaced with init() and still remains in that form now.
Functionally, the biggest change from pre-L340872 was introduced a bit later - PassIDCountMap that counts instances and adds #instance-numbers to the report.
And it remains "unreverted" in this change.


Repository:
  rL LLVM

https://reviews.llvm.org/D52356





More information about the llvm-commits mailing list