[PATCH] D126747: [BOLT] Add support for GOTPCRELX relocations

Rafael Auler via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 1 19:29:53 PDT 2022


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

Thanks! LGTM pending the test fix



================
Comment at: bolt/lib/Target/X86/X86MCSymbolizer.cpp:91
+  if (Relocation && Relocation::isX86GOTPCRELX(Relocation->Type)) {
+    // If the operand if PC-relative, convert it without using the relocation
+    // information. For GOTPCRELX, it is safe to use the absolute address
----------------
If the operand if PC-relative ->  If the operand is PC-relative


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126747



More information about the llvm-commits mailing list