<div dir="ltr">Anton, what do you think of David's patch with this test case?  OK to commit that?</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Feb 20, 2013 at 12:43 PM, Anton Korobeynikov <span dir="ltr"><<a href="mailto:anton@korobeynikov.info" target="_blank">anton@korobeynikov.info</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The patch looks incorrect. The code just needs to handle \1 properly<br>
and clang extended to add explicit \1 to the names which does not<br>
require mangling.<br>
<br>
I do not think that moving whole mangling to clang is a good idea,<br>
because then everyone who uses LLVM to call WinApi functions will need<br>
to mangle by hands.<br>
<div><div class="h5"><br>
On Wed, Feb 20, 2013 at 11:25 PM, Timur Iskhodzhanov<br>
<<a href="mailto:timurrrr@google.com">timurrrr@google.com</a>> wrote:<br>
> I don't remember anything other that what I've written in the bug Joćo<br>
> has mentioned.<br>
><br>
> Probably something like this patch<br>
> <a href="http://llvm.org/bugs/show_bug.cgi?id=14410#c6" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=14410#c6</a><br>
> ?<br>
><br>
> 2013/2/20 Joćo Matos <<a href="mailto:ripzonetriton@gmail.com">ripzonetriton@gmail.com</a>>:<br>
>> I think so. There have been other reports lately related to this being<br>
>> wrong.<br>
>><br>
>> <a href="http://llvm.org/bugs/show_bug.cgi?id=14410" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=14410</a><br>
>><br>
>> CC'ing Timur since he might know more about this.<br>
>><br>
>> On Wed, Feb 20, 2013 at 5:27 PM, David Nadlinger <<a href="mailto:code@klickverbot.at">code@klickverbot.at</a>><br>
>> wrote:<br>
>>><br>
>>> On Tue, Feb 19, 2013 at 2:13 PM, Duncan Sands <<a href="mailto:baldrick@free.fr">baldrick@free.fr</a>> wrote:<br>
>>> >> My question: Is there an easy way of disabling the name-mangling part<br>
>>> >> but keep the rest of the CC that I missed?<br>
>>> > if you use "\1" + "usual name", it will disable name mangling if you are<br>
>>> > lucky.  A leading \1 is LLVM's way of saying: leave this name alone!<br>
>>><br>
>>> Seems like I'm out of luck - the @<n> suffix is added<br>
>>> (AddFastCallStdCallSuffix) in the GlobalValue<br>
>>> Magnler::getNameWithPrefix overload, without paying respect to whether<br>
>>> the name originally had a '\1' prefix or not.<br>
>>><br>
>>> Should this be changed?<br>
>>><br>
>>> David<br>
>>> _______________________________________________<br>
>>> LLVM Developers mailing list<br>
>>> <a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
>>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
>><br>
>><br>
>><br>
>><br>
>> --<br>
>> Joćo Matos<br>
><br>
> _______________________________________________<br>
> LLVM Developers mailing list<br>
> <a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
<br>
<br>
<br>
</div></div>--<br>
With best regards, Anton Korobeynikov<br>
Faculty of Mathematics and Mechanics, Saint Petersburg State University<br>
<div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
</div></div></blockquote></div><br></div>