This is an archive of the discontinued LLVM Phabricator instance.

[RISCV] Add SiFive cores to the CPU option
ClosedPublic

Authored by evandro on Oct 2 2020, 3:40 PM.

Details

Summary

Add the SiFive cores E76 and U74 using the SiFive 7 series microarchitecture.

Diff Detail

Event Timeline

evandro created this revision.Oct 2 2020, 3:40 PM
evandro requested review of this revision.Oct 2 2020, 3:40 PM
khchen accepted this revision.Oct 4 2020, 7:20 PM

LGTM.

This revision is now accepted and ready to land.Oct 4 2020, 7:20 PM
Jim added inline comments.Oct 4 2020, 7:46 PM
llvm/lib/Target/RISCV/RISCV.td
257

It should only have one blank line.

evandro marked an inline comment as done.Oct 5 2020, 1:27 PM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptOct 5 2020, 1:58 PM
thakis added a subscriber: thakis.Oct 5 2020, 5:35 PM

This seems to break tests: http://45.33.8.238/linux/29545/step_7.txt

Can you take a look and revert for now if it takes a while to fix?

jrtc27 added a comment.Oct 5 2020, 5:48 PM

This seems to break tests: http://45.33.8.238/linux/29545/step_7.txt

Can you take a look and revert for now if it takes a while to fix?

I see it should already have been fixed in a48d480e1f7ebc5d5f93507fe1f519496621e259.