[PATCH] D74756: [yaml2obj] - Change the default order if implicitly created sections.

[George Rimar via Phabricator via llvm-commits]

grimar created this revision.
grimar added reviewers: jhenderson, MaskRay.
Herald added subscribers: rupprecht, hiraditya, emaste.
Herald added a reviewer: espindola.
Herald added a reviewer: alexshap.
Herald added a reviewer: rupprecht.
Herald added a project: LLVM.

.dynsym and .dynstr are allocatable and therefore normally are placed
before non-allocatable .strtab, .shstrtab, .symtab sections.
But we are placing them after currently what creates a mix of
alloc/non-alloc sections and does not look normal.


https://reviews.llvm.org/D74756

Files:
  llvm/lib/ObjectYAML/ELFEmitter.cpp
  llvm/test/tools/llvm-objcopy/ELF/add-symbol-no-symtab.test
  llvm/test/tools/llvm-readobj/ELF/dynamic-reloc-no-section-headers.test
  llvm/test/tools/llvm-readobj/ELF/hidden-versym.test
  llvm/test/tools/llvm-readobj/ELF/reloc-symbol-with-versioning.test
  llvm/test/tools/llvm-readobj/ELF/verdef-invalid.test
  llvm/test/tools/llvm-readobj/ELF/versioninfo.test
  llvm/test/tools/llvm-readobj/ELF/versym-invalid.test
  llvm/test/tools/yaml2obj/ELF/dynamic-symbols.yaml
  llvm/test/tools/yaml2obj/ELF/gnu-hash-section.yaml
  llvm/test/tools/yaml2obj/ELF/implicit-sections-info.yaml
  llvm/test/tools/yaml2obj/ELF/implicit-sections-types.yaml

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D74756.245093.patch
Type: text/x-patch
Size: 13352 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200218/7bc0415e/attachment.bin>