This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Support fixed vector copysign.
ClosedPublic

Authored by craig.topper on Mar 10 2021, 10:58 PM.

Diff Detail

Event Timeline

craig.topper created this revision.Mar 10 2021, 10:58 PM
craig.topper requested review of this revision.Mar 10 2021, 10:58 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 10 2021, 10:58 PM
Herald added a subscriber: MaskRay. · View Herald Transcript
frasercrmck accepted this revision.Mar 11 2021, 1:15 AM

LGTM otherwise.

llvm/test/CodeGen/RISCV/rvv/fixed-vectors-fp.ll
419

It might be worthwhile adding some extra tests to cover the FCOPYSIGN dag combine which reorders fpext/fptrunc and fneg. It looks like it was written such that it "just works" but you never know.

This revision is now accepted and ready to land.Mar 11 2021, 1:15 AM
This revision was automatically updated to reflect the committed changes.