<div dir="ltr">Done in r233238<div><br></div><div>Thanks!<br><br>-eric<br></div></div><br><div class="gmail_quote">On Wed, Mar 25, 2015 at 5:41 PM Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_quote">On Wed, Mar 25, 2015 at 5:35 PM Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> +def mno_sse4 : Flag<["-"], "mno-sse4">, Alias<mno_sse4_1>;<br>
...<br>
> +def msse4 : Flag<["-"], "msse4">, Alias<msse4_2>;<br>
<br>
<br>
The "mno" variant alias 4.1 but the "m" variant alias 4.2? This should<br>
have a comment :-)<br>
<br></blockquote><div><br></div></div></div><div dir="ltr"><div class="gmail_quote"><div>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 :)</div><div><br></div><div>-eric</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Cheers,<br>
Rafael<br>
</blockquote></div></div></blockquote></div>