[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