[PATCH] D11194: Instantiate function declarations in instantiated functions.

Serge Pavlov sepavloff at gmail.com
Thu Jul 16 10:36:44 PDT 2015


sepavloff added inline comments.

================
Comment at: lib/Sema/SemaTemplateInstantiate.cpp:1695
@@ +1694,3 @@
+    } else if (OwningFunc->getLexicalDeclContext()->isFunctionOrMethod() &&
+               !OwningFunc->isThisDeclarationADefinition()) {
+      // This is a function declaration within a function definition, as in:
----------------
rsmith wrote:
> Is this necessary? A local function declaration can't be a definition.
As clang do not support nested functions, it is not necessary. Remove it.


http://reviews.llvm.org/D11194







More information about the cfe-commits mailing list