This is an archive of the discontinued LLVM Phabricator instance.

[ELF] - Repair dynsym-pie.s testcase.
ClosedPublic

Authored by grimar on Aug 21 2017, 8:27 AM.

Details

Summary

It was implemented by me in D18183 and
looks broken from begining, because visibility attributes
were not applied properly to symbols before this patch.

I also removed one of symbols from testcase because
it looks was a silly copypaste, not sure what I wanted to test initially
with it.

Diff Detail

Repository
rL LLVM

Event Timeline

grimar created this revision.Aug 21 2017, 8:27 AM
ruiu accepted this revision.Aug 22 2017, 11:33 AM

LGTM

This revision is now accepted and ready to land.Aug 22 2017, 11:33 AM
This revision was automatically updated to reflect the committed changes.