This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Remove ExtZvl enum from RISCVSubtarget. NFC
ClosedPublic

Authored by craig.topper on Apr 8 2022, 11:40 AM.

Details

Summary

Having an enum with names that contain the string representation
of their value doesn't add any value. We can just use the numbers.

Diff Detail

Event Timeline

craig.topper created this revision.Apr 8 2022, 11:40 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 8 2022, 11:40 AM
craig.topper requested review of this revision.Apr 8 2022, 11:40 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 8 2022, 11:40 AM
frasercrmck accepted this revision.Apr 11 2022, 2:29 AM

LGTM, makes sense.

This revision is now accepted and ready to land.Apr 11 2022, 2:29 AM
kito-cheng accepted this revision.Apr 11 2022, 2:32 AM
This revision was landed with ongoing or failed builds.Apr 11 2022, 10:15 AM
This revision was automatically updated to reflect the committed changes.