<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Aug 26, 2013 at 4:00 PM, 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="im">On Mon, Aug 26, 2013 at 3:46 PM, Manman Ren <<a href="mailto:manman.ren@gmail.com">manman.ren@gmail.com</a>> wrote:<br>

> In r189282 and r189283.<br>
><br>
> I will work on clang front-end to generate the unique identifier for<br>
> DICompositeType.<br>
<br>
</div>For each field that uses a DIType that may refer to a DICompositeType,<br>
we'll see two patches - one for the backend support (to do the<br>
necessary indirect access through a map lookup) and frontend support<br>
to emit a name. </blockquote><div><br></div><div>By frontend support, are you referring to the change to DIBuilder? DIBuilder will</div><div>inspect the DIType reference, and if it is a DICompositeType and the unique</div>
<div>identifier is non-null, DIBuilder will use the identifier instead of the MDNode itself.</div><div><br></div><div>Per our discussion via IRC, I thought clang will first generate unique identifier for</div><div>external C++ DICompositeTypes. The next step is to update the usage of a DIType</div>
<div>that may refer to a DICompositeType (with one use as a start).</div><div>  </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The backend patch can be committed prior to the<br>

frontend patch (though the frontend patch would be a nice way to<br>
generate the backend test to ensure it's what we intend to<br>
support/works end-to-end).<br>
<br>
It'd be good to see exactly one (a small one, where possible) use of<br>
this first so we can checkin the map creation & lookup plumbing with a<br>
single simple field to motivate/demonstrate/test it, then<br>
incrementally commit support for each field after that.<br></blockquote><div>Agreed.</div><div><br></div><div>Thanks,</div><div>Manman </div></div></div><div class="gmail_extra"><br></div></div>