[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