<div dir="ltr">Hi David B,<div><br></div><div>Didn't read your email before submitting my change. Does r189498 look okay to you?</div><div><br></div><div>Manman</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Wed, Aug 28, 2013 at 10:32 AM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">On Wed, Aug 28, 2013 at 2:01 AM, David Majnemer<br>
<<a href="mailto:david.majnemer@gmail.com">david.majnemer@gmail.com</a>> wrote:<br>
> On Tue, Aug 27, 2013 at 5:42 PM, Manman Ren <<a href="mailto:manman.ren@gmail.com">manman.ren@gmail.com</a>> wrote:<br>
>><br>
>><br>
>><br>
>><br>
>> On Tue, Aug 27, 2013 at 2:34 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br>
>>><br>
>>> On Tue, Aug 27, 2013 at 2:28 PM, Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>> wrote:<br>
>>> > Why are we using the Itanium mangler for uuidof exprs?  Can we not do<br>
>>> > that?<br>
>>><br>
>>> We have a test case (see<br>
>>> clang/test/CodeGenCXX/debug-info-template.cpp) because we seem to<br>
>>> support the language feature on itanium platforms. If we shouldn't<br>
>>> allow that language extension in that case, we should disable it &<br>
>>> move this test out to a separate case.<br>
>>><br>
>>> (+David Majnemer who implemented this test/debug info support)<br>
>>><br>
>>> > If not, feel free to grab the approach used in the MS mangler and<br>
>>> > pretend to<br>
>>> > mangle a global variable called _GUID_1234....<br>
>>><br>
>>> I'm OK-ish with this, but deferring to you & David if you want to<br>
>>> avoid these language extensions cropping up outside of win32<br>
>>> platforms.<br>
>><br>
>><br>
>> Hi David M,<br>
>><br>
>> Any comment on whether we should move it out to a separate case?<br>
><br>
><br>
> We have yet to define a vendor mangling for it so I wouldn't worry about<br>
> testing it when targeting the itanium ABI.<br>
<br>
</div></div>Are there places this could lead to a crash in the existing code?<br>
Should we disable the language feature on Itanium targets to ensure<br>
this doesn't happen? (& modify the test case so the GUID stuff is only<br>
tested on win32 ABI targets)<br>
</blockquote></div><br></div>