[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