This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Support fmaximum/fminimum for fp16 vector when only Zvfhmin enabled
AbandonedPublic

Authored by Jim on Aug 29 2023, 7:03 PM.

Details

Summary

This patch promotes fmaximum/fminimum for fp16 vector to float operation.

Diff Detail

Event Timeline

Jim created this revision.Aug 29 2023, 7:03 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 29 2023, 7:03 PM
Jim requested review of this revision.Aug 29 2023, 7:03 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 29 2023, 7:03 PM
llvm/lib/Target/RISCV/RISCVISelLowering.cpp
5503

nit: maybe it makes sense for this to go in lowerFMAXIMUM_FMINIMUM?

Jim marked an inline comment as done.Sep 25 2023, 10:28 PM
Jim added inline comments.
llvm/lib/Target/RISCV/RISCVISelLowering.cpp
5503

Others for promoting fp16 vector operation to float vector operation also put this checking here.
I think we can have another PR to change all of them.