This is an archive of the discontinued LLVM Phabricator instance.

[LoongArch] Add codegen support for part of conversion operations
ClosedPublic

Authored by SixWeining on Jun 7 2022, 4:41 AM.

Diff Detail

Event Timeline

SixWeining created this revision.Jun 7 2022, 4:41 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 7 2022, 4:41 AM
SixWeining requested review of this revision.Jun 7 2022, 4:41 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 7 2022, 4:41 AM
rengolin accepted this revision.Jun 7 2022, 5:38 AM

LGTM, super simple, too.

This revision is now accepted and ready to land.Jun 7 2022, 5:38 AM
SixWeining updated this revision to Diff 435070.Jun 8 2022, 1:15 AM

Update the testcase since the parent commit is updated just now.

xen0n accepted this revision.Jun 8 2022, 11:19 PM

For now many of the operations are sub-optimal, but I guess that's only because niceties like bstrpick.[wd] are still not ready yet. I already saw ori $dest, $zero, 0 being taken care of in the next patch, so let's take care of them later; the testcases are certainly useful.

Thanks!

SixWeining updated this revision to Diff 435764.Jun 9 2022, 7:02 PM

Remove the patch number in the title.

SixWeining retitled this revision from [LoongArch 4/n] Add codegen support for part of conversion operations to [LoongArch] Add codegen support for part of conversion operations.Jun 9 2022, 7:02 PM
SixWeining edited the summary of this revision. (Show Details)Jun 10 2022, 9:06 PM

adjust the dependency

SixWeining edited the summary of this revision. (Show Details)Jun 13 2022, 6:16 AM
This revision was landed with ongoing or failed builds.Jun 19 2022, 7:01 PM
This revision was automatically updated to reflect the committed changes.