[PATCH] D144211: [BOLT] computing raw branch count for yaml profiles

[Amir Ayupov via Phabricator via llvm-commits]

Amir accepted this revision.
Amir added a comment.
This revision is now accepted and ready to land.

LGTM with small nits



================
Comment at: bolt/lib/Core/BinaryFunction.cpp:474
     OS << "\n  Profile Acc : " << format("%.1f%%", ProfileMatchRatio * 100.0f);
+    OS << "\n  RawBranchCount : " << RawBranchCount;
   }
----------------



================
Comment at: bolt/lib/Profile/YAMLProfileReader.cpp:87-91
+  for (const yaml::bolt::BinaryBasicBlockProfile &YamlBB : YamlBF.Blocks) {
+    for (const yaml::bolt::SuccessorInfo &YamlSI : YamlBB.Successors) {
+      FuncRawBranchCount += YamlSI.Count;
+    }
+  }
----------------



================
Comment at: bolt/test/X86/branch-data.test:18
 CHECK:      Exec Count  : 199
+CHECK:      RawBranchCount  : 7689
 CHECK:    }
----------------



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D144211/new/

https://reviews.llvm.org/D144211