<div dir="ltr">On Wed, Aug 28, 2013 at 4:10 PM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="im">On Wed, Aug 28, 2013 at 3:47 PM, Eli Friedman <span dir="ltr"><<a href="mailto:eli.friedman@gmail.com" target="_blank">eli.friedman@gmail.com</a>></span> wrote:<br>
</div><div class="gmail_extra"><div class="gmail_quote"><div class="im">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">The concept generally seems fine.<div><br></div><div>The setLocalExternDecl thing seems a bit fiddly, since I'm pretty sure it can be derived from information we already keep around.  Did you consider computing that information on demand?</div>

</div></blockquote><div><br></div></div><div>Yes, that's certainly possible. It's convenient for this to be part of the IdentifierNamespace, and I don't want to complicate the hot path through name lookup with this special case any more than necessary, but I don't have any benchmarks to justify doing it this way.</div>
<div class="im">
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Do we compute the correct DeclContext for a friend in a local class which is a redeclaration of a local extern decl?  Might be nice to test.</div>

</div></blockquote><div><br></div></div><div>Yes, we do; I'll add a test for that.</div></div></div></div></blockquote><div><br></div><div>This turns out to be hard (impossible?) to observe, since a local friend must be a redeclaration of an entity in the innermost surrounding block scope, but I've added a test for this interaction in general.</div>
<div><span style="color:rgb(80,0,80)"> </span></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class="im">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>It looks like some changes slipped in which should be committed separately (around where you added the reference to 6.2.7p2).</div>

</div></blockquote><div><br></div></div><div>I separately committed the functional part of those changes, but I'll take another pass and see if there's more that I can split out here.</div></div></div></div></blockquote>
<div><br></div><div>Done.</div><div><span style="color:rgb(80,0,80)"> </span></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra">
<div class="gmail_quote"><div class="im"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div>Does you patch work correctly with an extern local decl inside a block?  e.g. "void f() { ^{ void extern_decl(); extern_decl(); }(); }"?</div></div></blockquote><div><br></div></div><div>Yes. I'll add a test for that, too.</div>
</div></div></div></blockquote><div><br></div><div>Done.</div></div></div></div>