This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Remove fcvt.d.l(u) and fcvt.l(u).d instructions with _IN32X suffix.
ClosedPublic

Authored by craig.topper on Jun 14 2023, 12:56 PM.

Details

Summary

This is the same as D152950 without depending on D152948.

_IN32X instructions are for Zdinx on RV32 where doubles are split
across 2 registers.

fcvt.d.l(u) and fcvt.l(u).d are RV64 only instructions so we don't
need _IN32X versions of them.

Diff Detail

Event Timeline

craig.topper created this revision.Jun 14 2023, 12:56 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 14 2023, 12:56 PM
craig.topper requested review of this revision.Jun 14 2023, 12:56 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 14 2023, 12:56 PM
sunshaoce accepted this revision.Jun 14 2023, 4:43 PM

LGTM. I think this patch can be landed first.

This revision is now accepted and ready to land.Jun 14 2023, 4:43 PM