================
@@ -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");
----------------
necto wrote:
@NagyDonat, are you speaking about grepping for the full
`"ExprEngine::removeDead"` rather than just `removeDead`? This is what I meant
by the third drawback, it is easier to find the explicit string literal. On the
other hand, I don't know how different that is from, for example, grepping for
just `removeDead` that would show up in the trace, which should also yield the
desired function. Given that `removeDead` function name is still there right
next to the `TimeScope` declaration, I consider it should be fine
discoverability-wise
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
