This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Recognize new relocation types
ClosedPublic

Authored by PkmX on Aug 8 2017, 3:26 AM.

Details

Summary

These new relocation types were taken from latest RISC-V binutils.

They are needed by the RISC-V port of lld which we plan to upstream, so we figured we should let this change land in llvm first.

Diff Detail

Repository
rL LLVM

Event Timeline

PkmX created this revision.Aug 8 2017, 3:26 AM
asb accepted this revision.Aug 8 2017, 6:43 AM

Many thanks, R_RISCV_32_PCREL isn't currently documented in the riscv-elf-psabi-doc, but it seems was applied to upstream binutils. Looks good to me.

This revision is now accepted and ready to land.Aug 8 2017, 6:43 AM
This revision was automatically updated to reflect the committed changes.
asb added a comment.Aug 15 2017, 5:14 AM

I've committed this patch for you, after verifying R_RISCV_32_PCREL has been included in a release binutils (and therefore can be considered "final" despite not being documented). I've made an issue on the riscv-elf-psabi-doc for documenting this relocation https://github.com/riscv/riscv-elf-psabi-doc/issues/36. Thanks again.