[PATCH] D78888: [llvm-objcopy][MachO] Fix symbol table

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 1 17:18:27 PDT 2020


MaskRay accepted this revision.
MaskRay added a comment.

LGTM as well, with a nit.



================
Comment at: llvm/test/tools/llvm-objcopy/MachO/symbol-table.test:23
+## Verify that the binary is valid and check its symbol table.
+# RUN: llvm-readobj -symbols %t.copy | FileCheck %s
+
----------------
`--symbols`

We prefer double-dash options for LLVM binary utilities.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78888





More information about the llvm-commits mailing list