This is an archive of the discontinued LLVM Phabricator instance.

[Arm][AArch64] Make getArchFeatures to use TargetParser.def
ClosedPublic

Authored by danielkiss on Sep 21 2022, 4:35 AM.

Diff Detail

Event Timeline

danielkiss created this revision.Sep 21 2022, 4:35 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 21 2022, 4:35 AM
danielkiss requested review of this revision.Sep 21 2022, 4:35 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 21 2022, 4:35 AM
DavidSpickett accepted this revision.Sep 21 2022, 6:28 AM

I'm assuming this passes check-all though I'm surprised something didn't break with the number of tests that use llvm-mc. We don't maintain the triple and feature names on the llvm side as an ABI of any sort, so as long as clang still accepts what it used to then fine with the extra "a"s.

Be prepared for a few random failures in other subprojects but hey that's what bots are for right.

This LGTM, glad to see this getting more automated.

This revision is now accepted and ready to land.Sep 21 2022, 6:28 AM

Thanks, the ARM::getArchSynonym handles the differences as I see so only those tests failed that string matched v8. check-all passed on my machine.

This revision was landed with ongoing or failed builds.Sep 23 2022, 1:25 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptSep 23 2022, 1:25 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript