On Tue, May 21, 2013 at 11:53 AM, Lawrence Crowl <span dir="ltr"><<a href="mailto:crowl@google.com" target="_blank">crowl@google.com</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">
<div class="im">On 5/20/13, Manuel Klimek <<a href="mailto:klimek@google.com">klimek@google.com</a>> wrote:<br>
> ================<br>
> Comment at: lib/Lex/PPDirectives.cpp:573-582<br>
> @@ +572,12 @@<br>
> +        ModuleMap::ModuleHeaderRole Role = SuggestedModule->getRole();<br>
> +        // // Check for consistency between the module header role<br>
> +        // // as obtained from the lookup and as obtained from the module.<br>
> +        // // This check is expensive, so enable it when needed.<br>
> +        // SmallVectorImpl<const FileEntry *> &PvtHdrs<br>
> +        //     = RequestedModule->PrivateHeaders;<br>
> +        // SmallVectorImpl<const FileEntry *>::iterator Look<br>
> +     //     = std::find(PvtHdrs.begin(), PvtHdrs.end(), FE);<br>
> +        // bool IsPrivate = Look != PvtHdrs.end();<br>
> +        // assert((IsPrivate && Role == ModuleMap::PrivateHeader)<br>
> +        //        || (!IsPrivate && Role != ModuleMap::PrivateHeader));<br>
> +        if (Role == ModuleMap::PrivateHeader) {<br>
> ----------------<br>
> This looks unintentionally left as a comment?<br>
<br>
</div>It was left intentionally, so that it could be enabled when needed.</blockquote><div><br></div><div>Please uncomment it and wrap in an #ifndef NDEBUG.</div></div>