<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Mar 24, 2014 at 4:18 PM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@gmail.com" target="_blank">echristo@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 Mon, Mar 24, 2014 at 4:12 PM, Manman Ren <<a href="mailto:manman.ren@gmail.com">manman.ren@gmail.com</a>> wrote:<br>

><br>
><br>
><br>
> On Fri, Mar 21, 2014 at 3:16 PM, Adrian Prantl <<a href="mailto:aprantl@apple.com">aprantl@apple.com</a>> wrote:<br>
>><br>
>> Author: adrian<br>
>> Date: Fri Mar 21 17:16:32 2014<br>
>> New Revision: 204528<br>
>><br>
>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=204528&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=204528&view=rev</a><br>
>> Log:<br>
>> Dwarf Debug: Remove some cargo-cult type uniquing. Scopes do not have<br>
>> an ID, so this is a noop.<br>
>> Thanks Manman for catching this!<br>
>><br>
>> Modified:<br>
>>     llvm/trunk/lib/CodeGen/AsmPrinter/DwarfDebug.cpp<br>
>><br>
>> Modified: llvm/trunk/lib/CodeGen/AsmPrinter/DwarfDebug.cpp<br>
>> URL:<br>
>> <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfDebug.cpp?rev=204528&r1=204527&r2=204528&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/AsmPrinter/DwarfDebug.cpp?rev=204528&r1=204527&r2=204528&view=diff</a><br>

>><br>
>> ==============================================================================<br>
>> --- llvm/trunk/lib/CodeGen/AsmPrinter/DwarfDebug.cpp (original)<br>
>> +++ llvm/trunk/lib/CodeGen/AsmPrinter/DwarfDebug.cpp Fri Mar 21 17:16:32<br>
>> 2014<br>
>> @@ -604,7 +604,7 @@ DIE *DwarfDebug::constructScopeDIE(Dwarf<br>
>>      return NULL;<br>
>><br>
>>    // Unique scope where applicable.<br>
>> -  DIScope DS(resolve(DIScope(Scope->getScopeNode()).getRef()));<br>
>> +  DIScope DS(Scope->getScopeNode());<br>
><br>
><br>
> Right now, DICompositeType has an ID and it is also one kind of DIScope, so<br>
> the statement "scopes do not have an ID" sounds incorrect :]<br>
><br>
> DISubprogram does not have an ID and we should remove the following noop:<br>
><br>
> /// getOrCreateSubprogramDIE - Create new DIE using SP.<br>
> DIE *DwarfUnit::getOrCreateSubprogramDIE(DISubprogram SP) {<br>
>   ...<br>
>   // Unique declarations based on the ODR, where applicable.<br>
>   SP = DISubprogram(DD->resolve(SP.getRef()));<br>
><br>
<br>
</div></div>Yeah, where are we on using references for all types? I thought when<br>
you'd finished that we used them everywhere</blockquote><div><br></div><div>I thought so too, but I apparently missed DI[Global]Variable (thanks to Adrian for fixing those). </div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
 but apparently not. We<br>
should get that fixed so there's less confusion here.<br></blockquote><div><br></div><div>Yes.</div><div><br></div><div>Thanks,</div><div>Manman </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<span class="HOEnZb"><font color="#888888"><br>
-eric<br>
</font></span></blockquote></div><br></div></div>