This is an archive of the discontinued LLVM Phabricator instance.

[llvm][AArch64] Actually check expected FPU for CPUs
ClosedPublic

Authored by DavidSpickett on Dec 16 2020, 3:01 AM.

Details

Summary

We were passing this as an argument but never using
it. ARM has always checked this.

Note that the FPU list is shared between ARM and AArch64
so there is no AArch64::getFPUName, just ARM::getFPUName.

Diff Detail

Event Timeline

DavidSpickett created this revision.Dec 16 2020, 3:01 AM
DavidSpickett requested review of this revision.Dec 16 2020, 3:01 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 16 2020, 3:01 AM
dmgreen accepted this revision.Dec 16 2020, 10:35 AM
dmgreen added a subscriber: dmgreen.

Sounds good to me.

This revision is now accepted and ready to land.Dec 16 2020, 10:35 AM
This revision was landed with ongoing or failed builds.Dec 17 2020, 1:16 AM
This revision was automatically updated to reflect the committed changes.