[PATCH] D63280: [llvm-objdump] Use <first-symbol>-<offset> as the section start symbol

[Yuanfang Chen via Phabricator via llvm-commits]

ychen updated this revision to Diff 205189.
ychen added a comment.
Herald added a reviewer: alexshap.

- Add the code and test to show that the dummy section start symbol should not be used to resolve branch target in disassembly.
- Update test and some comments.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63280

Files:
  lld/test/ELF/aarch64-condb-reloc.s
  lld/test/ELF/aarch64-feature-bti.s
  lld/test/ELF/aarch64-feature-btipac.s
  lld/test/ELF/aarch64-feature-pac.s
  lld/test/ELF/aarch64-gnu-ifunc-plt.s
  lld/test/ELF/aarch64-ifunc-bti.s
  lld/test/ELF/aarch64-thunk-pi.s
  lld/test/ELF/aarch64-tstbr14-reloc.s
  lld/test/ELF/gnu-ifunc-noplt-i386.s
  lld/test/ELF/gnu-ifunc-noplt.s
  lld/test/ELF/gnu-ifunc-plt-i386.s
  lld/test/ELF/gnu-ifunc-plt.s
  lld/test/ELF/gnu-ifunc-shared.s
  lld/test/ELF/plt-aarch64.s
  lld/test/ELF/plt-i686.s
  lld/test/ELF/plt.s
  lld/test/ELF/ppc64-relocs.s
  lld/test/ELF/startstop-gccollect.s
  llvm/test/MC/AMDGPU/branch-comment.s
  llvm/test/MC/X86/branch-comment.s
  llvm/test/Object/Inputs/no-start-symbol.elf-x86_64
  llvm/test/Object/X86/no-start-symbol.test
  llvm/test/tools/llvm-objdump/X86/elf-disassemble-dynamic-symbols.test
  llvm/test/tools/llvm-objdump/X86/elf-disassemble-no-start-symbol.test
  llvm/test/tools/llvm-objdump/X86/elf-disassemble-symbol-labels-rel.test
  llvm/tools/llvm-objdump/llvm-objdump.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63280.205189.patch
Type: text/x-patch
Size: 27854 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190617/cecf4c81/attachment-0001.bin>