This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Merge fp-imm.ll and zfh-imm.ll into float/double/half-imm.ll. NFC
ClosedPublic

Authored by craig.topper on Aug 2 2023, 11:35 AM.

Details

Summary

fp-imm.ll and zfh-imm.ll test 0.0 and -0.0 while float/double/half-imm.ll
tested other non-zero constants. It seems like they should all be
tested together.

There are slight coverage changes due to different command lines,
but I'm not sure its meaningful. For example, we now don't test
double 0.0 and -0.0 with only the F extension.

Diff Detail

Event Timeline

craig.topper created this revision.Aug 2 2023, 11:35 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 2 2023, 11:35 AM
craig.topper requested review of this revision.Aug 2 2023, 11:35 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 2 2023, 11:35 AM
Herald added subscribers: eopXD, MaskRay. · View Herald Transcript
asb accepted this revision.Aug 2 2023, 12:24 PM

LGTM.

This revision is now accepted and ready to land.Aug 2 2023, 12:24 PM
This revision was landed with ongoing or failed builds.Aug 2 2023, 8:21 PM
This revision was automatically updated to reflect the committed changes.
llvm/test/CodeGen/RISCV/half-imm.ll