<div dir="ltr">I think those names look better than "dont-use-movt".<br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jul 9, 2015 at 8:53 AM, Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
> On 2015-Jul-08, at 16:16, Akira Hatanaka <<a href="mailto:ahatanak@gmail.com">ahatanak@gmail.com</a>> wrote:<br>
<span class="">><br>
><br>
>> On 2015-Jul-08, at 13:52, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br>
>><br>
>> Let's have the feature be a positive feature that just happens to be on by default in a lot of places. +/-dont-use-movt seems weird.<br>
>><br>
</span><span class="">> Do you mean I should replace ARMSubtarget::DontUseMovt with ARMSubtarget::UseMovt which is true by default? That should work.<br>
><br>
> Or do you mean we should use FeatureUseMovt : SubtargetFeature<"use-movt"> instead of FeatureDontUseMovt<"dont-use-movt">? If I understand correctly how subtarget features work, I think that would require clang (and other front-ends) to add feature "+use-movt" whenever we want to allow emitting movt/movw pairs as opposed to adding "+dont-use-movt" only when we want to disallow doing so. Assuming you would want to allow movt/movw pairs in the common case, wouldn't it be better to use a negative feature?<br>
<br>
</span>Maybe there's another negative name that isn't as strange.  Something<br>
like "avoid-movt" or "no-movt"?</blockquote></div><br></div></div>