[llvm] Reapply [BOLT] DataAggregator support for binaries with multiple text segments (PR #118023)

via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 28 09:01:15 PST 2024


github-actions[bot] wrote:

<!--LLVM CODE FORMAT COMMENT: {clang-format}-->


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

<details>
<summary>
You can test this locally with the following command:
</summary>

``````````bash
git-clang-format --diff a9b3ec154d7ab2d0896ac5c9f1e9a1266a37be80 fbc1b8aeacf2272338e50507d3f0f93378f85f49 --extensions h,cpp -- bolt/unittests/Core/MemoryMaps.cpp bolt/include/bolt/Profile/DataAggregator.h bolt/lib/Profile/DataAggregator.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/bolt/unittests/Core/MemoryMaps.cpp b/bolt/unittests/Core/MemoryMaps.cpp
index 2efaffedb9..230fd31414 100644
--- a/bolt/unittests/Core/MemoryMaps.cpp
+++ b/bolt/unittests/Core/MemoryMaps.cpp
@@ -136,6 +136,7 @@ TEST_P(MemoryMapsTester, MultipleSegmentsMismatchedBaseAddress) {
 
   DataAggregator DA("");
   BC->setFilename(Filename);
-  ASSERT_DEBUG_DEATH({ Error Err = DA.preprocessProfile(*BC); },
-                     "Base address on multiple segment mappings should match");
+  ASSERT_DEBUG_DEATH(
+      { Error Err = DA.preprocessProfile(*BC); },
+      "Base address on multiple segment mappings should match");
 }

``````````

</details>


https://github.com/llvm/llvm-project/pull/118023


More information about the llvm-commits mailing list