This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Separate the Zfhmin and Zfh extensions.
ClosedPublic

Authored by craig.topper on Jan 30 2022, 10:10 PM.

Details

Summary

The spec doesn't seem to be written as if Zfh implies Zfhmin. They
seem to be separate extensions.

This patch moves the instructions from Zfhmin to be enabled with
either the Zfh or Zfhmin extensions.

Diff Detail

Event Timeline

craig.topper created this revision.Jan 30 2022, 10:10 PM
craig.topper requested review of this revision.Jan 30 2022, 10:10 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 30 2022, 10:10 PM

LGTM. I think this is indeed a better way to implement Zfhmin.

This revision is now accepted and ready to land.Jan 31 2022, 4:33 AM
This revision was landed with ongoing or failed builds.Jan 31 2022, 9:08 AM
This revision was automatically updated to reflect the committed changes.