This is an archive of the discontinued LLVM Phabricator instance.

[ELF] Update tests to work even with modified defaults
ClosedPublic

Authored by phosek on Mar 14 2017, 8:21 PM.

Details

Summary

D30229 changes the defaults based on section names to match the GAS behavior, which breaks some of the tests that rely on the old defaults.

Diff Detail

Repository
rL LLVM

Event Timeline

phosek created this revision.Mar 14 2017, 8:21 PM
ruiu accepted this revision.Mar 14 2017, 8:28 PM

LGTM

test/ELF/merge-reloc.s
71 ↗(On Diff #91813)

LLD does not accept writable mergeable sections (*). Even though this test does not depend on that limitation (because you are passing the -r option), it doesn't make much sense to pass an object file that is considered invalid by LLD.

You want to change this to .rodata instead to keep the original intention of this test.

This revision is now accepted and ready to land.Mar 14 2017, 8:28 PM
ruiu requested changes to this revision.Mar 14 2017, 8:28 PM
This revision now requires changes to proceed.Mar 14 2017, 8:28 PM
phosek updated this revision to Diff 91815.Mar 14 2017, 8:34 PM
phosek edited edge metadata.
phosek marked an inline comment as done.
ruiu added inline comments.Mar 14 2017, 8:37 PM
test/ELF/merge-reloc.s
51 ↗(On Diff #91815)

This

55 ↗(On Diff #91815)

this

85 ↗(On Diff #91815)

and this are unnecessary changes, no?

phosek updated this revision to Diff 91816.Mar 14 2017, 8:42 PM
phosek marked 3 inline comments as done.
ruiu accepted this revision.Mar 14 2017, 8:44 PM

LGTM

This revision is now accepted and ready to land.Mar 14 2017, 8:44 PM
This revision was automatically updated to reflect the committed changes.