[PATCH] D134349: [Arm][AArch64] Make getArchFeatures to use TargetParser.def
David Spickett via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 21 06:28:17 PDT 2022
DavidSpickett accepted this revision.
DavidSpickett added a comment.
This revision is now accepted and ready to land.
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.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D134349/new/
https://reviews.llvm.org/D134349
More information about the llvm-commits
mailing list