[PATCH] D113734: [llvm-objcopy][MachO] Add llvm-strip support for newer load commands

Fri Nov 12 00:05:27 PST 2021

jhenderson added a comment.

In D113734#3126419 <https://reviews.llvm.org/D113734#3126419>, @keith wrote:

> I'm not sure how this should be tested since there aren't cases where we actually link binaries before being able to run this

I'm not too familiar with the Mach-O side of llvm-objcopy, but a quick search shows that at least some (possibly all) of the existing load commands are tested using yaml2obj-generated inputs. Take a look at the eixsting tests like lc-load-weak-dylib.test. If yaml2obj support needs to be extended for the new load commands, it should be fairly simple to do so.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113734