<div dir="ltr"><div>-llvm-dev, +llvm-commits</div><div><br></div>Any thoughts on this?</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Mar 29, 2013 at 4:37 AM, 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">Reid,<br>
<br>
I believe I had some concerns over the patch... Let me find / resurrect them :)<br>
<div class="HOEnZb"><div class="h5"><br>
On Fri, Mar 29, 2013 at 4:12 AM, Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>> wrote:<br>
> Anton, what do you think of David's patch with this test case?  OK to commit<br>
> that?<br>
><br>
><br>
> On Wed, Feb 20, 2013 at 12:43 PM, Anton Korobeynikov<br>
> <<a href="mailto:anton@korobeynikov.info">anton@korobeynikov.info</a>> wrote:<br>
>><br>
>> 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>
>><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>><br>
>> >>> wrote:<br>
>> >>> >> My question: Is there an easy way of disabling the name-mangling<br>
>> >>> >> 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<br>
>> >>> > 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>
>> --<br>
>> With best regards, Anton Korobeynikov<br>
>> Faculty of Mathematics and Mechanics, Saint Petersburg State University<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>
<br>
<br>
--<br>
With best regards, Anton Korobeynikov<br>
Faculty of Mathematics and Mechanics, Saint Petersburg State University<br>
</div></div></blockquote></div><br></div>