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

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.