[llvm-branch-commits] [BOLT][NFCI] Simplify DataAggregator using traces (PR #138181)

[via llvm-branch-commits]

github-actions[bot] wrote:




:warning: C/C++ code formatter, clang-format found issues in your code. 
:warning:



You can test this locally with the following command:


``bash
git-clang-format --diff HEAD~1 HEAD --extensions h,cpp -- 
bolt/include/bolt/Profile/DataAggregator.h bolt/lib/Profile/DataAggregator.cpp
``





View the diff from clang-format here.


``diff
diff --git a/bolt/lib/Profile/DataAggregator.cpp 
b/bolt/lib/Profile/DataAggregator.cpp
index 4e42e0d80..f74cf0264 100644
--- a/bolt/lib/Profile/DataAggregator.cpp
+++ b/bolt/lib/Profile/DataAggregator.cpp
@@ -1469,7 +1469,7 @@ void DataAggregator::processBranchEvents() {
   NamedRegionTimer T("processBranch", "Processing branch events",
  TimerGroupName, TimerGroupDesc, opts::TimeAggregator);
 
-  for (const auto &[Trace, Info]: Traces) {
+  for (const auto &[Trace, Info] : Traces) {
 if (Trace.From != Trace.To)
   doBranch(Trace.From, Trace.To, Info.TakenCount, Info.MispredCount);
 if (Trace.FallthroughEnd)

``




https://github.com/llvm/llvm-project/pull/138181
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

[llvm-branch-commits] [BOLT][NFCI] Simplify DataAggregator using traces (PR #138181)

[Amir Ayupov via llvm-branch-commits]

https://github.com/aaupov created 
https://github.com/llvm/llvm-project/pull/138181

Trace is a triple of (branch source, branch target, fallthrough end),
introduced in https://github.com/llvm/llvm-project/pull/127125.

Use traces throughout branch profile handling, unifying perf parsing
and pre-aggregated format handling. This in turn makes it possible to
use pre-aggregated data for heatmaps.

Test Plan: NFCI



___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits