llvmorg-github-actions[bot] wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang-temporal-safety @llvm/pr-subscribers-clang Author: Gábor Horváth (Xazax-hun) <details> <summary>Changes</summary> The TimeTraceScope was constructed as a temporary and destroyed immediately, so the prepass was reported as taking ~0. It actually accounts for ~14% of LoanPropagation in some cases. --- Full diff: https://github.com/llvm/llvm-project/pull/213521.diff 1 Files Affected: - (modified) clang/lib/Analysis/LifetimeSafety/LoanPropagation.cpp (+1-1) ``````````diff diff --git a/clang/lib/Analysis/LifetimeSafety/LoanPropagation.cpp b/clang/lib/Analysis/LifetimeSafety/LoanPropagation.cpp index 078892bd48c10..12b88c18b90b4 100644 --- a/clang/lib/Analysis/LifetimeSafety/LoanPropagation.cpp +++ b/clang/lib/Analysis/LifetimeSafety/LoanPropagation.cpp @@ -29,7 +29,7 @@ namespace clang::lifetimes::internal { // referenced in more than one basic block. static llvm::BitVector computePersistentOrigins(const FactManager &FactMgr, const CFG &C) { - llvm::TimeTraceScope("ComputePersistentOrigins"); + llvm::TimeTraceScope TimeProfile("ComputePersistentOrigins"); unsigned NumOrigins = FactMgr.getOriginMgr().getNumOrigins(); llvm::BitVector PersistentOrigins(NumOrigins); `````````` </details> https://github.com/llvm/llvm-project/pull/213521 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
