[PATCH] D58508: [ELF][test]Remove unnecessary empty symbol references in yaml/add missing symbols for relocs
James Henderson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 21 07:40:20 PST 2019
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>
More information about the llvm-commits
mailing list