This is an archive of the discontinued LLVM Phabricator instance.

[AARCH64] Add support for Broadcom Vulcan
ClosedPublic

Authored by pgode on Jun 19 2016, 1:47 AM.

Details

Summary

Adding core tuning support for new Broadcom Vulcan core (ARMv8.1A).

Diff Detail

Repository
rL LLVM

Event Timeline

pgode updated this revision to Diff 61191.Jun 19 2016, 1:47 AM
pgode retitled this revision from to [AARCH64] Add support for Broadcom Vulcan.
pgode updated this object.
pgode added reviewers: t.p.northover, rengolin.
pgode added subscribers: llvm-commits, MatzeB, aemerson, echristo.
rengolin added inline comments.Jun 20 2016, 3:03 AM
lib/Target/AArch64/AArch64.td
244 ↗(On Diff #61191)

Isn't it even remotely similar to an A53/A57?

lib/Target/AArch64/AArch64Subtarget.cpp
74 ↗(On Diff #61191)

Nit: This could be a fall-through for all remaining cases...

pgode marked 2 inline comments as done.Jun 20 2016, 3:17 AM
pgode added inline comments.
lib/Target/AArch64/AArch64.td
244 ↗(On Diff #61191)

This processor is similar to A57. This patch is for introducing Vulcan in LLVM.
We plan to submit the Vulcan scheduler model in subsequent patches, hence 'NoSchedModel.

lib/Target/AArch64/AArch64Subtarget.cpp
74 ↗(On Diff #61191)

We plan to add Vulcan specific details in subsequent patches under this case. Please let me know if that is ok.

rengolin accepted this revision.Jun 20 2016, 3:20 AM
rengolin edited edge metadata.

Makes sense. LGTM, thanks!

This revision is now accepted and ready to land.Jun 20 2016, 3:20 AM
This revision was automatically updated to reflect the committed changes.
mloud added a subscriber: mloud.Sep 25 2021, 11:39 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 25 2021, 11:39 AM