This is an archive of the discontinued LLVM Phabricator instance.

[ELF][MIPS] Fix .MIPS.options ri_gp_value on MIPS64
ClosedPublic

Authored by arichardson on Dec 12 2016, 4:57 AM.

Details

Summary

The VA of _gp was being truncated to 32 bits when calling getVa() but
for 64bit MIPS we need to write a 64 bit value to .MIPS.options

Diff Detail

Repository
rL LLVM

Event Timeline

arichardson retitled this revision from to [ELF][MIPS] Fix .MIPS.options ri_gp_value on MIPS64.
arichardson updated this object.
arichardson added reviewers: atanasyan, ruiu.
atanasyan accepted this revision.Dec 12 2016, 6:01 AM
atanasyan edited edge metadata.

LGTM. Thanks!

As to me I would add a test case.

Do you have a commit access? If not, I could commit the patch and add the test case by myself.

This revision is now accepted and ready to land.Dec 12 2016, 6:01 AM

No, I don't have commit access. It would be great if you could commit it and add the test case. Thanks!

This revision was automatically updated to reflect the committed changes.