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

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

Thanks Renato for taking the time to provide such a detailed response. The case is not as clear cut as I initially thought. I'll have to think about it a bit more.

In the meantime, I created https://reviews.llvm.org/D35826, which adds a error message to Clang's driver if -marm/-mno-thumb is used in combination with M-profile -mcpu/-march options, as it seems there's agreement that that's a thing we should definitely check in the frontend. I'll also plan to follow this up with a patch that errors/warns if the -mcpu/-march options are incompatible.


https://reviews.llvm.org/D35569