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

[Richard Barton via Phabricator via llvm-commits]

richard.barton.arm added a comment.

Agreed with what Florian says. My understanding of -target is that it is used to set up which flavour of toolchain we are using and using it to set the architecture is a shorthand for saying '-target=arm... -march=armvX'. I also expect it to be somewhat like GCC where the 'arm-...' triples cover ARM/AArch32 architecture in general and not the ARM ISA per se.

So I would expect -target, -march and -mcpu to set the architecture and override eachother in that order, i.e mcpu > march > target, then -marm|-mthumb set the ISA. We warn if the ISA is incompatible with the architecture.


https://reviews.llvm.org/D35569