[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