<div dir="ltr">Sure that looks reasonable. Might want to add a comment that we need the imported modules bit to run last since it depends on the entire compile unit die being constructed first.<div><br></div><div>And tests. But you knew that :)<br>
<div><br></div><div>-eric</div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Mar 27, 2013 at 4:24 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">Based on our IRC conversation this morning I've worked up the basics<br>
of DW_TAG_imported_module support by using a CU-level list rather than<br>
one list per scope. While this still has a few rough edges<br>
(DIImportedModule should expand to include file/line information as<br>
well to be consistent with GCC, and my Clang changes (not attached<br>
here) don't seem to account for global using directives but they work<br>
for function local ones).<br>
<br>
Does this seem about right? Better/worse/indifferent compared to<br>
per-scope lists? (seems better to me, at least)<br>
<br>
If so, I'll revert r178099 (which added the extra field to every<br>
DIScope with the intent that that would contain the list of<br>
DIImportedModules for each scope) & continue down this path.<br>
</blockquote></div><br></div>