This is an archive of the discontinued LLVM Phabricator instance.

[Object] Support relocation resolver for CSKY
ClosedPublic

Authored by zixuan-wu on May 12 2022, 12:37 AM.

Details

Summary

It enables relocation resolver for CSKY to pass some check-all test, and also add some test about dwarf relocs.

Diff Detail

Event Timeline

zixuan-wu created this revision.May 12 2022, 12:37 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 12 2022, 12:37 AM
zixuan-wu requested review of this revision.May 12 2022, 12:37 AM
Herald added a project: Restricted Project. · View Herald Transcript
zixuan-wu set the repository for this revision to rG LLVM Github Monorepo.May 12 2022, 12:43 AM
zixuan-wu edited reviewers, added: asb, mstorsjo; removed: jdoerfert.May 12 2022, 12:46 AM

Is there any more part of code or test I should develop in this patch together?

MaskRay added inline comments.May 12 2022, 1:01 PM
llvm/test/DebugInfo/CSKY/eh-frame.s
2
2

Add a file-level comment about the purpose of the test.

MaskRay accepted this revision.May 12 2022, 1:42 PM
MaskRay added inline comments.
llvm/lib/Object/RelocationResolver.cpp
502

Remove the variable.

This revision is now accepted and ready to land.May 12 2022, 1:42 PM
MaskRay added a comment.EditedMay 12 2022, 1:43 PM

[CSKY] Enable relocation resolver for CSKY

Perhaps [Object] Support relocation resolver for CSKY.

"Enable" conveys that something has been already implemented.

zixuan-wu retitled this revision from [CSKY] Enable relocation resolver for CSKY to [Object] Support relocation resolver for CSKY.May 12 2022, 8:04 PM
zixuan-wu updated this revision to Diff 429132.May 12 2022, 8:41 PM

Address comments.

This revision was landed with ongoing or failed builds.May 12 2022, 10:39 PM
This revision was automatically updated to reflect the committed changes.