[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