[PATCH] D61718: [llvm-objdump] Print st_other

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 10 03:26:02 PDT 2019


jhenderson added inline comments.


================
Comment at: llvm/test/tools/llvm-objdump/elf-symbol-visibility.s:1
+# REQUIRES: powerpc-registered-target
+# RUN: llvm-mc -filetype=obj -triple=powerpc64-pc-freebsd13.0 %s -o %t
----------------
I think it's more normal to put a test with such a requirement inside a PowerPC/X86 subdirectory for llvm-objdump tests, but does this need to be assembly? Can it not be built with yaml2obj?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D61718





More information about the llvm-commits mailing list