This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Define vnsrl/vnsra intrinsics.
ClosedPublic

Authored by HsiangKai on Dec 14 2020, 6:07 AM.

Details

Summary

Define vnsrl/vnsra 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, 6:07 AM
HsiangKai requested review of this revision.Dec 14 2020, 6:08 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 14 2020, 6:08 AM
Herald added a subscriber: MaskRay. · View Herald Transcript

This looks good to me other than my previous comment on another review about Wideable->Widenable

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

Update test cases.

HsiangKai updated this revision to Diff 311833.Dec 15 2020, 1:18 AM

Address @craig.topper's comments.

This revision is now accepted and ready to land.Dec 15 2020, 11:22 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.