[all-commits] [llvm/llvm-project] 277998: [yaml2obj][obj2yaml] - Do not create a symbol tabl...

[llvm-git-migration via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 2779987d0e2985a9618bd87b5f6532722645a5f2
      https://github.com/llvm/llvm-project/commit/2779987d0e2985a9618bd87b5f6532722645a5f2
  Author: George Rimar <grimar at accesssoftek.com>
  Date:   2019-10-20 (Sun, 20 Oct 2019)

  Changed paths:
    M llvm/include/llvm/ObjectYAML/ELFYAML.h
    M llvm/lib/ObjectYAML/ELFEmitter.cpp
    M llvm/test/Object/invalid.test
    M llvm/test/Object/objdump-sectionheaders.test
    M llvm/test/tools/llvm-objcopy/ELF/add-section-remove.test
    M llvm/test/tools/llvm-objcopy/ELF/add-section.test
    M llvm/test/tools/llvm-objcopy/ELF/add-symbol.test
    M llvm/test/tools/llvm-objcopy/ELF/basic-only-section.test
    M llvm/test/tools/llvm-objcopy/ELF/explicit-keep-remove.test
    M llvm/test/tools/llvm-objcopy/ELF/explicit-only-section-remove.test
    M llvm/test/tools/llvm-objcopy/ELF/invalid-e_shoff.test
    M llvm/test/tools/llvm-objcopy/ELF/keep-only-section.test
    M llvm/test/tools/llvm-objcopy/ELF/no-strip-all.test
    M llvm/test/tools/llvm-objcopy/ELF/no-symbol-relocation.test
    M llvm/test/tools/llvm-objcopy/ELF/null-symbol.test
    M llvm/test/tools/llvm-objcopy/ELF/only-section-many.test
    M llvm/test/tools/llvm-objcopy/ELF/preserve-segment-contents-ehdr-phdrs.test
    M llvm/test/tools/llvm-objcopy/ELF/remove-section.test
    M llvm/test/tools/llvm-objcopy/ELF/rename-section-multiple.test
    M llvm/test/tools/llvm-objcopy/ELF/rename-section.test
    M llvm/test/tools/llvm-objcopy/ELF/segment-shift-section-remove.test
    M llvm/test/tools/llvm-objcopy/ELF/segment-test-remove-section.test
    M llvm/test/tools/llvm-objcopy/ELF/shstrtab-optimize.test
    M llvm/test/tools/llvm-objcopy/ELF/strip-unneeded-remove-debug-keep-link.test
    M llvm/test/tools/llvm-objcopy/ELF/symtab-error-on-remove-strtab.test
    M llvm/test/tools/llvm-objcopy/ELF/symtab-link.test
    M llvm/test/tools/llvm-objcopy/ELF/wildcard-syntax.test
    M llvm/test/tools/llvm-objdump/full-contents.test
    M llvm/test/tools/llvm-objdump/relocations-elf.test
    M llvm/test/tools/llvm-readobj/check-output-order.test
    M llvm/test/tools/llvm-readobj/elf-dynamic-malformed.test
    M llvm/test/tools/llvm-readobj/elf-file-headers.test
    M llvm/test/tools/llvm-readobj/elf-hidden-versym.test
    M llvm/test/tools/llvm-readobj/elf-invalid-shstrndx.test
    M llvm/test/tools/llvm-readobj/elf-invalid-versioning.test
    M llvm/test/tools/llvm-readobj/elf-no-phdrs.test
    M llvm/test/tools/llvm-readobj/elf-no-relocs.test
    M llvm/test/tools/llvm-readobj/elf-relr-relocs.test
    M llvm/test/tools/llvm-readobj/elf-versioninfo.test
    M llvm/test/tools/llvm-readobj/elf-wrong-shstrtab-type.test
    M llvm/test/tools/llvm-readobj/reloc-types-elf-aarch64.test
    M llvm/test/tools/llvm-readobj/reloc-types-elf-arm.test
    M llvm/test/tools/llvm-readobj/reloc-types-elf-lanai.test
    M llvm/test/tools/llvm-readobj/reloc-types-elf-mips.test
    M llvm/test/tools/llvm-readobj/reloc-types-elf-mips64.test
    M llvm/test/tools/obj2yaml/elf-llvm-addrsig-section.yaml
    A llvm/test/tools/obj2yaml/elf-no-symtab.yaml
    M llvm/test/tools/obj2yaml/elf-sht-symtab-shndx.yaml
    M llvm/test/tools/obj2yaml/no-symbol-reloc.test
    M llvm/test/tools/obj2yaml/section-group.test
    M llvm/test/tools/yaml2obj/dynamic-symbols.yaml
    M llvm/test/tools/yaml2obj/elf-comdat-broken-info.yaml
    M llvm/test/tools/yaml2obj/elf-custom-null-section.yaml
    M llvm/test/tools/yaml2obj/elf-header-sh-fields.yaml
    M llvm/test/tools/yaml2obj/elf-sht-symtab-shndx.yaml
    M llvm/test/tools/yaml2obj/implicit-sections-types.test
    M llvm/test/tools/yaml2obj/implicit-sections.test
    M llvm/test/tools/yaml2obj/invalid-symboless-relocation.yaml
    M llvm/test/tools/yaml2obj/reloc-sec-info.yaml
    M llvm/test/tools/yaml2obj/symboless-relocation.yaml
    M llvm/test/tools/yaml2obj/symtab-implicit-sections-flags.yaml
    M llvm/tools/obj2yaml/elf2yaml.cpp

  Log Message:
  -----------
  [yaml2obj][obj2yaml] - Do not create a symbol table by default.

This patch tries to resolve problems faced in D68943
and uses some of the code written by Konrad Wilhelm Kleine
in that patch.

Previously, yaml2obj tool always created a .symtab section.
This patch changes that. With it we only create it when
have a "Symbols:" tag in the YAML document or when
we need to create it because it is used by another section(s).

obj2yaml follows the new behavior and does not print "Symbols:"
anymore when there is no symbol table.

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

llvm-svn: 375361