================ @@ -1031,6 +1032,7 @@ void ExprEngine::removeDead(ExplodedNode *Pred, ExplodedNodeSet &Out, const LocationContext *LC, const Stmt *DiagnosticStmt, ProgramPoint::Kind K) { + llvm::TimeTraceScope TimeScope("ExprEngine::removeDead"); ---------------- steakhal wrote:
I think if it's unique, it's a good fit. I don't think we strictly need the namespace part or the function parameter types in the name. I think the shorter the better - assuming they still remain grep-able and unique. https://github.com/llvm/llvm-project/pull/125884 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits