[PATCH] D66734: [llvm-readobj/llvm-readelf] - Report a proper warning when dumping a broken dynamic relocation.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 26 06:07:59 PDT 2019


MaskRay added inline comments.


================
Comment at: test/tools/llvm-readobj/elf-broken-dynamic-reloc-name.test:31
+    Info:    0
+    Address: 0x0
+    Relocations:
----------------
`Address` and `Info` can be deleted.


================
Comment at: test/tools/llvm-readobj/elf-broken-dynamic-reloc-name.test:50
+ProgramHeaders:
+  - Type:  PT_LOAD
+    Sections:
----------------
Is the PT_LOAD used?


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

https://reviews.llvm.org/D66734





More information about the llvm-commits mailing list