[clang] [analyzer] Refine TimeTrace name for dispatchWorkItem (PR #128352)
Balazs Benics via cfe-commits
cfe-commits at lists.llvm.org
Sat Feb 22 03:43:53 PST 2025
https://github.com/steakhal created https://github.com/llvm/llvm-project/pull/128352
Fixes https://github.com/llvm/llvm-project/pull/125508#discussion_r1965038954
>From ff647449a2eb0c5ac4930e86e53c02a04ac78ec3 Mon Sep 17 00:00:00 2001
From: Balazs Benics <benicsbalazs at gmail.com>
Date: Sat, 22 Feb 2025 12:40:31 +0100
Subject: [PATCH] [analyzer] Refine TimeTrace name for dispatchWorkItem
Fixes https://github.com/llvm/llvm-project/pull/125508#discussion_r1965038954
---
clang/lib/StaticAnalyzer/Core/CoreEngine.cpp | 2 +-
clang/test/Analysis/ftime-trace.cpp | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/clang/lib/StaticAnalyzer/Core/CoreEngine.cpp b/clang/lib/StaticAnalyzer/Core/CoreEngine.cpp
index bf1fd7c2356dc..d96211c3a6635 100644
--- a/clang/lib/StaticAnalyzer/Core/CoreEngine.cpp
+++ b/clang/lib/StaticAnalyzer/Core/CoreEngine.cpp
@@ -183,7 +183,7 @@ bool CoreEngine::ExecuteWorkList(const LocationContext *L, unsigned MaxSteps,
static std::string timeTraceScopeName(const ProgramPoint &Loc) {
if (llvm::timeTraceProfilerEnabled()) {
- return llvm::formatv("Loc {0}",
+ return llvm::formatv("dispatchWorkItem {0}",
ProgramPoint::getProgramPointKindName(Loc.getKind()))
.str();
}
diff --git a/clang/test/Analysis/ftime-trace.cpp b/clang/test/Analysis/ftime-trace.cpp
index 2c369a9bf781e..2940ff2e02891 100644
--- a/clang/test/Analysis/ftime-trace.cpp
+++ b/clang/test/Analysis/ftime-trace.cpp
@@ -23,7 +23,7 @@
// The trace also contains durations of each step, but they are so short that they are not reliably present
// in each run. However, they are also aggregated into Total *, for example:
//
-// CHECK: "name": "Total Loc PostStmt",
+// CHECK: "name": "Total dispatchWorkItem PostStmt",
// CHECK-NEXT: "args": {
// CHECK-NEXT: "count": {{[0-9]+}},
// CHECK-NEXT: "avg ms": {{[0-9]+}}
More information about the cfe-commits
mailing list