This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Add Uses=[FRM] and mayRaiseFPException to VF(N/W)CVT instructions.
ClosedPublic

Authored by craig.topper on Aug 27 2022, 9:48 AM.

Diff Detail

Event Timeline

craig.topper created this revision.Aug 27 2022, 9:48 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 27 2022, 9:48 AM
craig.topper requested review of this revision.Aug 27 2022, 9:48 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 27 2022, 9:48 AM
kito-cheng accepted this revision.Aug 28 2022, 7:11 PM
This revision is now accepted and ready to land.Aug 28 2022, 7:11 PM

I think you're missing a "to" in the subject that significantly changes its meaning?

craig.topper retitled this revision from [RISCV] Add Uses=[FRM] and mayRaiseFPException VF(N/W)CVT instructions. to [RISCV] Add Uses=[FRM] and mayRaiseFPException to VF(N/W)CVT instructions..Aug 28 2022, 7:43 PM
arcbbb accepted this revision.Aug 29 2022, 2:11 AM

This patch is needed before D122860 & D123371 are good enough and useful.
LGTM.