[PATCH] D64788: [llvm-readelf] - A fix for: "--hash-symbols asserts for 64-bit ELFs"

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 18 05:59:08 PDT 2019


jhenderson added inline comments.


================
Comment at: test/tools/llvm-readobj/elf-hash-symbols.test:228
 
+# RUN: yaml2obj --docnum=4 %s -o %t2-64.so
+# RUN: llvm-readelf --hash-symbols %t2-64.so \
----------------
I don't think you need 64-bit versions of the "ONLY-HASH/ONLY-GNUHASH" cases, as long as there's one for the main case.


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

https://reviews.llvm.org/D64788





More information about the llvm-commits mailing list