<div dir="ltr">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><div class="gmail_extra"><div class="gmail_quote">
<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></blockquote><div><br></div><div>It shouldn't lead to a crash, the Itanium mangler will just issue an error diagnostic saying it couldn't mangle it.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
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><div class="gmail_extra">I don't know what the original intention of the adding __uuidof() to clang. The feature still works as long as you don't need to use it in a way that demands a mangling. There is some precedent for this as the GNU extension ?: with the middle operand omitted cannot be mangled on Itanium.</div>
</div>