This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Add new V instructions and aliases in v1.0-08a0b46.
ClosedPublic

Authored by HsiangKai on Dec 20 2020, 11:23 PM.

Details

Summary

New V instructions:
vfrsqrte7.v
vfrece7.v
vrgatherei16.vv

New aliases:
vneg.v
vncvt.x.x.w
vfneg.v

Diff Detail

Event Timeline

HsiangKai created this revision.Dec 20 2020, 11:23 PM
HsiangKai requested review of this revision.Dec 20 2020, 11:23 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 20 2020, 11:23 PM
Herald added a subscriber: MaskRay. · View Herald Transcript
HsiangKai edited the summary of this revision. (Show Details)Dec 20 2020, 11:31 PM
craig.topper added inline comments.Dec 20 2020, 11:50 PM
llvm/lib/Target/RISCV/RISCVInstrInfoV.td
805

Do you know if vfabs.v has been discussed for the V spec? We have a fabs.s alias for scalar. Seems odd that its not in the vector spec.

kito-cheng added inline comments.Dec 21 2020, 2:26 AM
llvm/lib/Target/RISCV/RISCVInstrInfoV.td
805

I sent PR to spec before but not merge yet.
https://github.com/riscv/riscv-v-spec/pull/574

craig.topper accepted this revision.Dec 21 2020, 9:59 AM

LGTM. But please title this "[RISCV] Add new V instructions and aliases in v1.0-08a0b46." and clarify in the commit message which ones are aliases.

This revision is now accepted and ready to land.Dec 21 2020, 9:59 AM
HsiangKai retitled this revision from [RISCV] Add new V instructions in v1.0-08a0b46. to [RISCV] Add new V instructions and aliases in v1.0-08a0b46..Dec 21 2020, 2:12 PM
HsiangKai edited the summary of this revision. (Show Details)