Page MenuHomePhabricator

[jitlink] add R_RISCV_BRANCH to jitlink
ClosedPublic

Authored by fourdim on Mon, Jan 3, 8:15 PM.

Details

Summary

This patch supported the R_RISCV_BRANCH relocation.

Diff Detail

Event Timeline

fourdim created this revision.Mon, Jan 3, 8:15 PM
fourdim requested review of this revision.Mon, Jan 3, 8:15 PM
Herald added a project: Restricted Project. · View Herald TranscriptMon, Jan 3, 8:15 PM
fourdim updated this revision to Diff 397787.Wed, Jan 5, 10:18 PM

Add prefix for hexadecimal numeric.

fourdim updated this revision to Diff 397788.Wed, Jan 5, 10:37 PM

Include the recent commits.

fourdim updated this revision to Diff 397789.Wed, Jan 5, 10:49 PM

Include the recent commits.

fourdim updated this revision to Diff 397810.Thu, Jan 6, 12:52 AM

Fix the behavior after extractBits function's update.

StephenFan added inline comments.
llvm/lib/ExecutionEngine/JITLink/ELF_riscv.cpp
167

More concise.

fourdim updated this revision to Diff 397833.Thu, Jan 6, 3:02 AM

Address the comments.

lhames accepted this revision.Thu, Jan 13, 6:27 PM

LGTM! Thanks very much @fourdim!

This revision is now accepted and ready to land.Thu, Jan 13, 6:27 PM
fourdim updated this revision to Diff 399889.Thu, Jan 13, 10:06 PM
fourdim marked an inline comment as done.

Include the recent commits.

This revision was automatically updated to reflect the committed changes.