This is an archive of the discontinued LLVM Phabricator instance.

Bug fix in AArch64 PLT Entry
ClosedPublic

Authored by stewartd on Sep 17 2014, 4:56 AM.

Details

Summary

This fixes an incorrect PLT entry.

Diff Detail

Event Timeline

stewartd updated this revision to Diff 13778.Sep 17 2014, 4:56 AM
stewartd retitled this revision from to Bug fix in AArch64 PLT Entry.
stewartd updated this object.
stewartd edited the test plan for this revision. (Show Details)
stewartd added a project: lld.
stewartd added a subscriber: Unknown Object (MLST).
mcrosier removed a subscriber: mcrosier.Sep 18 2014, 6:42 AM

The testcases that you have added with this commit, is nothing to do with the change that you have made for PLT.

I would recommend adding a test for the PLT0 change with this commit and separate the tests that you have created into a separate commit.

stewartd updated this revision to Diff 14014.Sep 23 2014, 1:19 PM
stewartd updated this object.

Removing the other test cases.

The got atom should have the name to the PLT0 atom in debug mode.

stewartd updated this revision to Diff 14035.Sep 24 2014, 5:20 AM

Adding the name of the PLT0 atom in debug mode.

shankar.easwaran edited edge metadata.
This revision is now accepted and ready to land.Sep 24 2014, 7:17 AM
mcrosier closed this revision.Sep 24 2014, 3:02 PM

Committed in r218418.