This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Add intrinsics for vf[n]macc/vf[n]msac/vf[n]madd/vf[n]msub instructions
ClosedPublic

Authored by arcbbb on Dec 22 2020, 5:04 AM.

Details

Summary

This patch defines vfmadd/vfnmacc, vfmsac/vfnmsac, vfmadd/vfnmadd,
and vfmsub/vfnmsub lower to V instructions.

Authored-by: Roger Ferrer Ibanez <rofirrim@gmail.com>
Co-Authored-by: ShihPo Hung <shihpo.hung@sifive.com>

Diff Detail

Event Timeline

arcbbb created this revision.Dec 22 2020, 5:04 AM
arcbbb requested review of this revision.Dec 22 2020, 5:04 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 22 2020, 5:04 AM
This revision is now accepted and ready to land.Dec 22 2020, 10:57 AM
jrtc27 added inline comments.Dec 22 2020, 11:10 AM
llvm/lib/Target/RISCV/RISCVInstrInfoVPseudos.td
890–891

!if(IsFloat, "_VF", "_VX") (or with !not if you really want it the other way round)?

jrtc27 added inline comments.Dec 22 2020, 11:12 AM
llvm/lib/Target/RISCV/RISCVInstrInfoVPseudos.td
1959

TableGen has true/false these days that's better than 0/1 when using a bit like this. Though I see the file currently already uses the old style in a bunch of places.