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

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 9 04:21:42 PDT 2017


fhahn added a comment.

In https://reviews.llvm.org/D35569#836330, @echristo wrote:

> In https://reviews.llvm.org/D35569#816289, @rengolin wrote:
>
> > Correct. `FeatureNoARM` is specific to M-class. I don't mind how we call it as long as it's explicit the intention, and I think both `FeatureNoARM` and `FeatureThumbOnly` are clear.
>
>
> Well, clear or not is a bit subjective ;)


Agreed, I think there's no need to change that.

> That said, I think this is a strict improvement.

Great, thanks for having another look. I'll commit the change soon soon.


https://reviews.llvm.org/D35569





More information about the llvm-commits mailing list