[all-commits] [llvm/llvm-project] e4ceb8: [lib/ObjectYAML] - Make `ELFYAML::Relocation::Offs...

Georgii Rymar via All-commits all-commits at lists.llvm.org
Fri Mar 6 03:09:58 PST 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: e4ceb8f4217d9203ecd7134d5fe2bac6d49c724d
      https://github.com/llvm/llvm-project/commit/e4ceb8f4217d9203ecd7134d5fe2bac6d49c724d
  Author: Georgii Rymar <grimar at accesssoftek.com>
  Date:   2020-03-06 (Fri, 06 Mar 2020)

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

  Log Message:
  -----------
  [lib/ObjectYAML] - Make `ELFYAML::Relocation::Offset` optional.

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.

Differential revision: https://reviews.llvm.org/D75608




More information about the All-commits mailing list