This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Add RISCVProcFamilyEnum and add SiFive7.

Authored by craig.topper on Jan 11 2022, 3:48 PM.



Use it to remove explicit string compares from unrolling preferences.

I'm of two minds on this. Ideally, we would define things in terms
of architectural or microarchitectural features, but it's hard to
do that with things like unrolling preferences without just ending up
with FeatureSiFive7UnrollingPreferences.

Having a proc enum is consistent with ARM and AArch64. X86 only has
a few and is trying to move away from it.

Diff Detail

Event Timeline

craig.topper created this revision.Jan 11 2022, 3:48 PM
craig.topper requested review of this revision.Jan 11 2022, 3:48 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 11 2022, 3:48 PM
Herald added a subscriber: MaskRay. · View Herald Transcript
mcberg2021 accepted this revision.Jan 11 2022, 4:49 PM

Nice and its extensible too for future targets under the query function with processor model mapping the family grouping. By and large LGTM. Lets see what folks think...

This revision is now accepted and ready to land.Jan 11 2022, 4:49 PM


asb accepted this revision.Jan 12 2022, 1:17 AM
This revision was landed with ongoing or failed builds.Jan 12 2022, 9:34 AM
This revision was automatically updated to reflect the committed changes.