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

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 18 08:07:37 PDT 2019


grimar 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 \
----------------
jhenderson wrote:
> 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.
Removed.


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

https://reviews.llvm.org/D64788





More information about the llvm-commits mailing list