[PATCH] D118526: [llvm-objcopy][MachO] Ignore LC_LINKER_OPTION when redefining symbols

Keith Smiley via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 31 15:50:04 PST 2022


This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG7a836ba3c5f2: [llvm-objcopy][MachO] Ignore LC_LINKER_OPTION when redefining symbols (authored by keith).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118526

Files:
  llvm/test/tools/llvm-objcopy/MachO/redefine-symbol.s
  llvm/tools/llvm-objcopy/MachO/MachOLayoutBuilder.cpp


Index: llvm/tools/llvm-objcopy/MachO/MachOLayoutBuilder.cpp
===================================================================
--- llvm/tools/llvm-objcopy/MachO/MachOLayoutBuilder.cpp
+++ llvm/tools/llvm-objcopy/MachO/MachOLayoutBuilder.cpp
@@ -417,6 +417,7 @@
     case MachO::LC_SUB_UMBRELLA:
     case MachO::LC_SUB_CLIENT:
     case MachO::LC_SUB_LIBRARY:
+    case MachO::LC_LINKER_OPTION:
       // Nothing to update.
       break;
     default:
Index: llvm/test/tools/llvm-objcopy/MachO/redefine-symbol.s
===================================================================
--- llvm/test/tools/llvm-objcopy/MachO/redefine-symbol.s
+++ llvm/test/tools/llvm-objcopy/MachO/redefine-symbol.s
@@ -43,6 +43,7 @@
 # RUN: llvm-objcopy --strip-symbol _ba --redefine-sym _foo=_ba %t %t.noba
 # RUN: llvm-readobj --symbols %t.noba | FileCheck %s --check-prefix=NOTSTRIPPED
 
+.linker_option "-lc"
 .globl _func
 _func:
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D118526.404759.patch
Type: text/x-patch
Size: 917 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220131/22dea386/attachment.bin>


More information about the llvm-commits mailing list