This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Order the RISCVInstrInfo*.td includes for standard extensions into logical groups. NFC
ClosedPublic

Authored by craig.topper on Jul 19 2023, 9:58 PM.

Details

Summary

There are some ordering dependency between these files.

-F must be included before D, Zfh, and Zfa. I recently suggested Zfbfmin
should be after Zfh.
-V must be before Zvk.
-Zc must be after Zb since Zbb instructions can be compressed.

So I've grouped all the scalar FP together. The vector together.
And put the compressed instructions at the end.

Diff Detail

Event Timeline

craig.topper created this revision.Jul 19 2023, 9:58 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 19 2023, 9:58 PM
craig.topper requested review of this revision.Jul 19 2023, 9:58 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 19 2023, 9:58 PM
Herald added subscribers: eopXD, MaskRay. · View Herald Transcript
wangpc accepted this revision.Jul 19 2023, 11:59 PM

LGTM.

This revision is now accepted and ready to land.Jul 19 2023, 11:59 PM
asb accepted this revision.Jul 20 2023, 3:20 AM

LGTM

This revision was landed with ongoing or failed builds.Jul 20 2023, 8:14 AM
This revision was automatically updated to reflect the committed changes.