This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Implement the vneg.v builtin.
ClosedPublic

Authored by HsiangKai on Apr 20 2021, 12:36 AM.

Diff Detail

Event Timeline

HsiangKai created this revision.Apr 20 2021, 12:36 AM
HsiangKai requested review of this revision.Apr 20 2021, 12:36 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 20 2021, 12:36 AM
frasercrmck added inline comments.Apr 21 2021, 1:54 AM
clang/utils/TableGen/RISCVVEmitter.cpp
1048

Does this only affect vneg or are there other changes to the tablegenned code as a consequence?

khchen added inline comments.Apr 21 2021, 3:03 AM
clang/test/CodeGen/RISCV/rvv-intrinsics-overloaded/vneg.c
4

nit: The integer intrinsic functions do not need the -target-feature +f, +d and +experimental-zfh.

craig.topper added inline comments.Apr 21 2021, 9:03 AM
clang/utils/TableGen/RISCVVEmitter.cpp
1048

Is this needed for fneg? When we implemented this change downstream, I thought it was for compares due to how vmsge.vx was handled.

HsiangKai updated this revision to Diff 339452.Apr 21 2021, 7:56 PM

Address comments.

HsiangKai marked 3 inline comments as done.Apr 21 2021, 7:57 PM
HsiangKai added inline comments.
clang/utils/TableGen/RISCVVEmitter.cpp
1048

Thanks for pointing it out. I reverted it.

This revision is now accepted and ready to land.Apr 22 2021, 4:45 PM
This revision was landed with ongoing or failed builds.Apr 22 2021, 8:41 PM
This revision was automatically updated to reflect the committed changes.
HsiangKai marked an inline comment as done.