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

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

We want to make sure "-mcpu=cortex-m0" implies -mthumb in clang... but yes, we probably don't want to mess with the target features in the backend.

In terms of cleaning this up, we should probably have "FeatureARM", for CPUs which support ARM-mode execution, rather than "FeatureNoARM"; negative features are confusing.


https://reviews.llvm.org/D35569