[PATCH] D35569: [ARM] Remove FeatureNoARM implies ModeThumb.

[Renato Golin via Phabricator via llvm-commits]

rengolin accepted this revision.
rengolin added a comment.
This revision is now accepted and ready to land.

Sorry, long holidays... :)

I agree with your interpretation of the ACLE and with the fact that llvm-mc and llc should behave accordingly.

Can you add a simple test to see that we do get the error when selecting -mtriple=arm -mcpu=m0?

With the added negative test, LGTM. Thanks!

cheers,
--renato


https://reviews.llvm.org/D35569