On Wed, Mar 25, 2015 at 5:35 PM Rafael EspĂ­ndola <[email protected]>
wrote:

> > +def mno_sse4 : Flag<["-"], "mno-sse4">, Alias<mno_sse4_1>;
> ...
> > +def msse4 : Flag<["-"], "msse4">, Alias<msse4_2>;
>
>
> The "mno" variant alias 4.1 but the "m" variant alias 4.2? This should
> have a comment :-)
>
>
That's fair. I commented in the commit and I think in the test, but I can
throw it in the code too. It's... subtle. And annoying really :)

-eric


> Cheers,
> Rafael
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to