This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Define vminu/vmin/vmaxu/vmax intrinsics.
ClosedPublic

Authored by HsiangKai on Dec 14 2020, 7:42 AM.

Details

Summary

Define vminu/vmin/vmaxu/vmax intrinsics and lower to V instructions.

We work with @rogfer01 from BSC to come out this patch.

Authored-by: Roger Ferrer Ibanez <rofirrim@gmail.com>
Co-Authored-by: Hsiangkai Wang <kai.wang@sifive.com>

Diff Detail

Event Timeline

HsiangKai created this revision.Dec 14 2020, 7:42 AM
HsiangKai requested review of this revision.Dec 14 2020, 7:42 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 14 2020, 7:42 AM
Herald added a subscriber: MaskRay. · View Herald Transcript

This seems like good to me

rogfer01 added a comment.EditedDec 14 2020, 11:59 PM

Agreed, these instructions and their intrinsics seems pretty uncontroversial. LGTM.

HsiangKai edited the summary of this revision. (Show Details)

Update test cases.

frasercrmck accepted this revision.Dec 15 2020, 1:50 AM
This revision is now accepted and ready to land.Dec 15 2020, 1:50 AM
This revision was landed with ongoing or failed builds.Dec 15 2020, 2:32 PM
This revision was automatically updated to reflect the committed changes.