This is an archive of the discontinued LLVM Phabricator instance.

[Timers] Fix printing some `-ftime-report` sections twice. Fixes PR40328.
ClosedPublic

Authored by vsapsai on Sep 17 2019, 4:05 PM.

Details

Summary

Starting from r324788 timer groups aren't cleared automatically when
printed out. As a result some timer groups were printed one more time.
For example, "Pass execution timing report" was printed again in
ManagedStatic<PassTimingInfo> destructor, "DWARF Emission" in
ManagedStatic<Name2PairMap> NamedGroupedTimers destructor.

Fix by clearing timer groups manually.

Diff Detail

Repository
rL LLVM

Event Timeline

vsapsai created this revision.Sep 17 2019, 4:05 PM
thegameg accepted this revision.Sep 17 2019, 4:17 PM

LGTM, thanks!

This revision is now accepted and ready to land.Sep 17 2019, 4:17 PM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptSep 17 2019, 5:04 PM

Thanks for the fast review.