[PATCH] D58508: [ELF][test]Remove unnecessary empty symbol references in yaml/add missing symbols for relocs

[James Henderson via Phabricator via llvm-commits]

jhenderson created this revision.
jhenderson added reviewers: ruiu, grimar, jakehehrlich.
Herald added subscribers: arichardson, javed.absar, emaste.
Herald added a reviewer: espindola.
Herald added a project: LLVM.

yaml2obj used to require the Symbol field in relocations, but it hasn't done so for a couple of years. I am making a separate change to yaml2obj that will look up the symbol by name or index, if present, and emit an error if not found. This will mean that an explicit symbol reference (even to an empty-named symbol) that does not reference a symbol declared in the yaml will result in an error.

This patch updates tests that would otherwise start emitting errors.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D58508

Files:
  test/ELF/arm-v4bx.test
  test/ELF/icf10.test
  test/ELF/icf11.test
  test/ELF/invalid/reloc-section-reordered.test
  test/ELF/relocation-dtrace.test
  test/ELF/relocation-none-aarch64.test
  test/ELF/relocation-none-i686.test
  test/ELF/undef-broken-debug.test

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D58508.187795.patch
Type: text/x-patch
Size: 3401 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190221/6433f3f1/attachment.bin>