[clang] [analyzer] Add -ftime-trace scopes for region-store bindings and removeDead (PR #125884)

[Arseniy Zaostrovnykh via cfe-commits]

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