Prefixing the the SubArch with plus sign makes the ArchFeature name.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
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.
Comment Actions
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.