[clang] [analyzer] Refine TimeTrace name for dispatchWorkItem (PR #128352)
via cfe-commits
cfe-commits at lists.llvm.org
Sat Feb 22 03:44:12 PST 2025
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-clang-static-analyzer-1
Author: Balazs Benics (steakhal)
<details>
<summary>Changes</summary>
Fixes https://github.com/llvm/llvm-project/pull/125508#discussion_r1965038954
---
Full diff: https://github.com/llvm/llvm-project/pull/128352.diff
2 Files Affected:
- (modified) clang/lib/StaticAnalyzer/Core/CoreEngine.cpp (+1-1)
- (modified) clang/test/Analysis/ftime-trace.cpp (+1-1)
``````````diff
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]+}}
``````````
</details>
https://github.com/llvm/llvm-project/pull/128352
More information about the cfe-commits
mailing list