I added some text in r205223.<br><div><br></div><div>-- </div><div>David Majnemer</div><br><div>On Mon Mar 31 2014 at 9:18:47 AM, Jordan Rose <<a href="mailto:jordan_rose@apple.com">jordan_rose@apple.com</a>> wrote:</div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Can you put that in a comment? Right now it's a bit opaque.<br>
<br>
Jordan<br>
<br>
On Mar 29, 2014, at 23:34 , David Majnemer <<a href="mailto:david.majnemer@gmail.com" target="_blank">david.majnemer@gmail.com</a>> wrote:<br>
<br>
> Author: majnemer<br>
> Date: Sun Mar 30 01:34:26 2014<br>
> New Revision: 205128<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=205128&view=rev" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project?rev=205128&view=rev</a><br>
> Log:<br>
> MS ABI: Simplify MangleByte<br>
><br>
> The delta between '\xe1' and '\xc1' is equivalent to the one between 'a'<br>
> and 'A'.  This allows us to reuse the computation between '\xe1' and<br>
> '\xfa' for the '\xc1' to '\xda' case.<br>
><br>
> No functionality change.<br>
><br>
> Modified:<br>
>    cfe/trunk/lib/AST/<u></u>MicrosoftMangle.cpp<br>
><br>
> Modified: cfe/trunk/lib/AST/<u></u>MicrosoftMangle.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/AST/MicrosoftMangle.cpp?rev=205128&r1=205127&r2=205128&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/cfe/trunk/lib/AST/<u></u>MicrosoftMangle.cpp?rev=<u></u>205128&r1=205127&r2=205128&<u></u>view=diff</a><br>

> ==============================<u></u>==============================<u></u>==================<br>
> --- cfe/trunk/lib/AST/<u></u>MicrosoftMangle.cpp (original)<br>
> +++ cfe/trunk/lib/AST/<u></u>MicrosoftMangle.cpp Sun Mar 30 01:34:26 2014<br>
> @@ -2421,9 +2421,8 @@ void MicrosoftMangleContextImpl::<u></u>mangleS<br>
>     if ((Byte >= 'a' && Byte <= 'z') || (Byte >= 'A' && Byte <= 'Z') ||<br>
>         (Byte >= '0' && Byte <= '9') || Byte == '_' || Byte == '$') {<br>
>       Mangler.getStream() << Byte;<br>
> -    } else if (Byte >= '\xe1' && Byte <= '\xfa') {<br>
> -      Mangler.getStream() << '?' << static_cast<char>('a' + (Byte - '\xe1'));<br>
> -    } else if (Byte >= '\xc1' && Byte <= '\xda') {<br>
> +    } else if ((Byte >= '\xe1' && Byte <= '\xfa') ||<br>
> +               (Byte >= '\xc1' && Byte <= '\xda')) {<br>
>       Mangler.getStream() << '?' << static_cast<char>('A' + (Byte - '\xc1'));<br>
>     } else {<br>
>       switch (Byte) {<br>
><br>
><br>
> ______________________________<u></u>_________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/cfe-commits</a><br>
<br>
</blockquote>