[llvm] [BOLT] Prevent .rela.plt reordering (PR #207412)

via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 3 08:24:35 PDT 2026


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 origin/main HEAD --extensions h,c,cpp -- bolt/test/runtime/X86/rela-plt-order.c bolt/include/bolt/Rewrite/RewriteInstance.h bolt/lib/Rewrite/RewriteInstance.cpp --diff_from_common_commit
``````````

:warning:
The reproduction instructions above might return results for more than one PR
in a stack if you are using a stacked PR workflow. You can limit the results by
changing `origin/main` to the base branch/commit you want to compare against.
:warning:

</details>

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

``````````diff
diff --git a/bolt/lib/Rewrite/RewriteInstance.cpp b/bolt/lib/Rewrite/RewriteInstance.cpp
index 359074ab0..9ac5b58a2 100644
--- a/bolt/lib/Rewrite/RewriteInstance.cpp
+++ b/bolt/lib/Rewrite/RewriteInstance.cpp
@@ -5891,12 +5891,11 @@ RewriteInstance::patchELFAllocatableRelaSections(ELFObjectFile<ELFT> *File) {
 
   {
     for (const uint64_t Address : DynamicRelocationOrder) {
-      ErrorOr<BinarySection &> SectionOrError = BC->getSectionForAddress(Address);
+      ErrorOr<BinarySection &> SectionOrError =
+          BC->getSectionForAddress(Address);
       if (!SectionOrError) {
-        BC->errs()
-          << "Cannot find section for dynamic relocation at 0x"
-          << Twine::utohexstr(Address)
-          << "\n";
+        BC->errs() << "Cannot find section for dynamic relocation at 0x"
+                   << Twine::utohexstr(Address) << "\n";
         exit(1);
       }
 
@@ -5906,7 +5905,8 @@ RewriteInstance::patchELFAllocatableRelaSections(ELFObjectFile<ELFT> *File) {
       if (!SectionAddress)
         SectionAddress = SectionInputAddress;
 
-      const Relocation &Rel = *Section.getDynamicRelocationAt(Address - SectionInputAddress);
+      const Relocation &Rel =
+          *Section.getDynamicRelocationAt(Address - SectionInputAddress);
 
       {
         const bool IsRelative = Rel.isRelative();

``````````

</details>


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


More information about the llvm-commits mailing list