<div dir="ltr">Please, commit for me.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 30, 2014 at 6:05 AM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Do you have commit access, or would you like me to commit this for you?</div><div class="HOEnZb"><div class="h5">
<div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 29, 2014 at 8:04 PM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">LGTM<br>
<br>
================<br>
Comment at: lib/AST/Decl.cpp:1307<br>
@@ +1306,3 @@<br>
+      if (P.SuppressUnwrittenScope &&<br>
+          (ND->isAnonymousNamespace() || ND->isInlineNamespace()))<br>
+        continue;<br>
----------------<br>
Just use `isInline` here since you already know you've got a namespace.<br>
<br>
<a href="http://reviews.llvm.org/D3922" target="_blank">http://reviews.llvm.org/D3922</a><br>
<br>
<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>