On Fri, Oct 5, 2012 at 2:21 AM, Nick Lewycky <span dir="ltr"><<a href="mailto:nicholas@mxc.ca" target="_blank">nicholas@mxc.ca</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
This patch changes UsingDirectives to be derived from Decl instead of NamedDecl, and consequently removes the CXXUsingDirective DeclarationName.<br>
<br>
The way using directives are stored with this patch is to have a single pointer on each UsingDirectiveDecl that points to the previous declaration, and a pointer on TranslationUnit and Namespace DeclContexts that point to the last one in that chain. We don't need such pointers on other DeclContexts because they either can't hold using-directives or the lookup will be handled by Scope.<br>

<br>
Please review!</blockquote><div><br></div><div>This looks good to me. Have you tested the patch with chained PCH (or modules)?</div></div>