This is an archive of the discontinued LLVM Phabricator instance.

[RISCV][NFC] Use named arguments in newly added changes
ClosedPublic

Authored by wangpc on Aug 9 2023, 8:48 PM.

Details

Summary

Named arguments have been supported in D152998 and most *.td files
of RISC-V target have been rewritten. This patch just makes them
consistent since some newly added changes weren't in this way.

Diff Detail

Event Timeline

wangpc created this revision.Aug 9 2023, 8:48 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 9 2023, 8:48 PM
wangpc requested review of this revision.Aug 9 2023, 8:48 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptAug 9 2023, 8:48 PM
wangpc edited the summary of this revision. (Show Details)Aug 9 2023, 8:49 PM
MaskRay accepted this revision.Aug 9 2023, 10:25 PM
This revision is now accepted and ready to land.Aug 9 2023, 10:25 PM
4vtomat accepted this revision.Aug 9 2023, 10:29 PM

LGTM, thanks~

asb accepted this revision.Aug 9 2023, 11:24 PM

I just wanted to say thank you for your work on the named arguments tablegen feature. This and D154067 are good showcases of the readability improvement.

This revision was landed with ongoing or failed builds.Aug 10 2023, 2:16 AM
This revision was automatically updated to reflect the committed changes.