<p dir="ltr">Agreed. </p>
<div class="gmail_quote">On Jun 27, 2013 12:06 AM, "Peter Collingbourne" <<a href="mailto:peter@pcc.me.uk">peter@pcc.me.uk</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Thu, Jun 27, 2013 at 08:41:12AM +0200, Duncan Sands wrote:<br>
> Hi Peter,<br>
><br>
> On 26/06/13 22:34, Peter Collingbourne wrote:<br>
>> On Wed, Jun 26, 2013 at 07:29:36PM +0200, Duncan Sands wrote:<br>
>>> Hi Peter,<br>
>>><br>
>>> On 26/06/13 01:59, Peter Collingbourne wrote:<br>
>>>> On Tue, Jun 25, 2013 at 04:57:38PM -0700, Peter Collingbourne wrote:<br>
>>>>> Hi,<br>
>>>>><br>
>>>>> This fixes debug info emission for std::nullptr_t (previously it would crash).<br>
>>>>><br>
>>>>> OK to commit?<br>
>>>><br>
>>>> With patch this time.<br>
>>><br>
>>> thanks.  How about getting the name from the type rather than assuming it is<br>
>>> nullptr_t?<br>
>><br>
>> I don't think it's possible for the debug info for nullptr_t to<br>
>> be of any form other than a DW_TAG_unspecified_type with a name of<br>
>> "nullptr_t" -- at least there doesn't seem to be a way to get Clang<br>
>> to spit out something else.  (For that matter, it isn't clear to me<br>
>> why DIBuilder::createNullPtrType lets you specify the name.)  Eric?<br>
><br>
> if other languages start to produce nullptr GCC nodes (and why not - maybe this<br>
> concept is useful for them too) I doubt that they will call it nullptr_t, so it<br>
> seems best to me to just take the GCC name.<br>
<br>
Sounds reasonable.  This patch follows GCC's logic when deciding the<br>
name of a nullptr type.  As a bonus, it also adds debug support for<br>
LANG_TYPE nodes.<br>
<br>
Thanks,<br>
--<br>
Peter<br>
</blockquote></div>