================
@@ -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

Reply via email to