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

Alexander Shaposhnikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 1 17:18:36 PDT 2020


alexshap marked an inline comment as done.
alexshap added inline comments.


================
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
+
----------------
MaskRay wrote:
> `--symbols`
> 
> We prefer double-dash options for LLVM binary utilities.
fair enough, will change before committing


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