[PATCH] D100489: [llvm-readobj] Fix dumping dynamic relative relocations for SHT_REL

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 19 06:19:00 PDT 2021


jhenderson added a comment.

The patch title tags llvm-readobj, but this is an llvm-objdump change.



================
Comment at: llvm/test/tools/llvm-objdump/X86/elf-dynamic-relocs-rel.test:1-3
+## Show that llvm-objdump can dump dynamic relocations for Elf_Rel relocations
+## Previously printing the R_386_RELATIVE relocations would trigger
+## error: can't read an entry at 0x40: it goes past the end of the section (0x40)
----------------
I don't think you need the "Previously ..." sentence at all. What you probably do want to say is that this test shows that Elf_Rel relocations can be dumped for both cases with and without a corresponding symbol, something like the inline edit.


================
Comment at: llvm/test/tools/llvm-objdump/X86/elf-dynamic-relocs-rel.test:9
+
+# CHECK:      file format elf32-i386
+# CHECK-LABEL: DYNAMIC RELOCATION RECORDS
----------------
I don't think this line is useful?


================
Comment at: llvm/test/tools/llvm-objdump/X86/elf-dynamic-relocs-rel.test:11-12
+# CHECK-LABEL: DYNAMIC RELOCATION RECORDS
+# CHECK-NEXT: 00000030 R_386_RELATIVE *ABS*
+# CHECK-NEXT: 00000010 R_386_NONE foo
+# CHECK-EMPTY:
----------------
Nit suggestion to make things line up nicely.


================
Comment at: llvm/test/tools/llvm-objdump/X86/elf-dynamic-relocs-rel.test:61-63
+Symbols:
+  - Name:    foo
+    Section: .data
----------------
I think you can omit the static symbol?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100489



More information about the llvm-commits mailing list