[PATCH] D51535: [PassTiming] reporting time-passes separately for multiple pass instances of the same pass
Serguei Katkov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Sep 2 19:00:46 PDT 2018
skatkov added a comment.
Could you please add a test producing something with #2 and #3... I guess any loop pass for function with 2-3 loops should work.
================
Comment at: lib/IR/PassTimingInfo.cpp:108
sys::SmartScopedLock<true> Lock(*TimingInfoMutex);
StringRef PassName = P->getPassName();
+ Timer *&T = TimingData[Pass];
----------------
Move to if() scope below. It is not used in this scope.
Repository:
rL LLVM
https://reviews.llvm.org/D51535
More information about the llvm-commits
mailing list