[PATCH] D75608: [lib/ObjectYAML] - Make `ELFYAML::Relocation::Offset` optional.

[George Rimar via Phabricator via llvm-commits]

grimar created this revision.
grimar added reviewers: jhenderson, MaskRay.
Herald added subscribers: kerbowa, rupprecht, arphaman, atanasyan, kbarton, hiraditya, nhaehnle, jvesely, nemanjai, emaste.
Herald added a reviewer: espindola.
Herald added a project: LLVM.

Currently `yaml2obj` require `Offset` field in a relocation description.
There are many cases when `Offset` is insignificant in a context of a test case.

Making `Offset` optional allows to simplify our test cases.
This is what this patch does.

Also, with this patch `obj2yaml` does not dump a zero offset of a relocation.


https://reviews.llvm.org/D75608

Files:
  lld/test/ELF/Inputs/shf-info-link.test
  lld/test/ELF/arm-thumb-undefined-weak-narrow.test
  lld/test/ELF/arm-v4bx.test
  lld/test/ELF/debug-relocation-none.test
  lld/test/ELF/i386-pc16.test
  lld/test/ELF/icf11.test
  lld/test/ELF/invalid/bad-reloc-target.test
  lld/test/ELF/invalid/invalid-debug-relocations.test
  lld/test/ELF/invalid/invalid-relocation-aarch64.test
  lld/test/ELF/invalid/invalid-relocation-x64.test
  lld/test/ELF/invalid/reloc-section-reordered.test
  lld/test/ELF/relocation-b-aarch64.test
  lld/test/ELF/relocation-group.test
  lld/test/ELF/shf-info-link.test
  lld/test/ELF/sht-group-empty.test
  lld/test/ELF/undef-broken-debug.test
  lld/test/ELF/undef-not-suggest.test
  llvm/lib/ObjectYAML/ELFYAML.cpp
  llvm/test/Object/AArch64/yaml2obj-elf-aarch64-rel.yaml
  llvm/test/Object/AMDGPU/elf64-relocs.yaml
  llvm/test/Object/Lanai/yaml2obj-elf-lanai-rel.yaml
  llvm/test/Object/X86/yaml-elf-x86-rel-broken.yaml
  llvm/test/Object/X86/yaml2obj-elf-x86-rel.yaml
  llvm/test/Object/invalid-alignment.test
  llvm/test/Object/obj2yaml.test
  llvm/test/Object/objdump-relocations.test
  llvm/test/tools/llvm-readobj/ELF/broken-dynamic-reloc.test
  llvm/test/tools/llvm-readobj/ELF/dynamic-reloc-no-section-headers.test
  llvm/test/tools/llvm-readobj/ELF/reloc-types-elf-aarch64.test
  llvm/test/tools/llvm-readobj/ELF/reloc-types-elf-arm.test
  llvm/test/tools/llvm-readobj/ELF/reloc-types-elf-lanai.test
  llvm/test/tools/llvm-readobj/ELF/reloc-types-elf-mips.test
  llvm/test/tools/llvm-readobj/ELF/reloc-types-elf-mips64.test
  llvm/test/tools/llvm-readobj/ELF/reloc-types-elf-ppc64.test
  llvm/test/tools/llvm-readobj/ELF/relocation-errors.test
  llvm/test/tools/llvm-readobj/ELF/relocations.test
  llvm/test/tools/obj2yaml/duplicate-symbol-and-section-names.test
  llvm/test/tools/obj2yaml/elf-ppc64-relocations.yaml
  llvm/test/tools/obj2yaml/no-symbol-reloc.test
  llvm/test/tools/yaml2obj/ELF/duplicate-symbol-names.yaml
  llvm/test/tools/yaml2obj/ELF/dynamic-relocations.yaml
  llvm/test/tools/yaml2obj/ELF/relocation-explicit-symbol-index.yaml
  llvm/test/tools/yaml2obj/ELF/relocation-missing-symbol.yaml
  llvm/test/tools/yaml2obj/ELF/symbol-name.yaml

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D75608.248159.patch
Type: text/x-patch
Size: 61450 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200304/941c6f0a/attachment.bin>