[PATCH] D44439: [Sema] Pop function scope when instantiating a func with skipped body

[Ilya Biryukov via Phabricator via cfe-commits]

ilya-biryukov added a comment.

I also came up with a test case, but it breaks because of invalid errors when function bodies are skipped. I'll make sure to include it along with a fix for the errors in a follow-up patch.



================
Comment at: lib/Sema/SemaTemplateInstantiateDecl.cpp:3942
       ActOnSkippedFunctionBody(Function);
+      // FIXME: finishing the function body while in an expression evaluation
+      // context seems wrong. Investigate more.
----------------
sammccall wrote:
> (hmm, not sure whether it's better to duplicate or not-duplicate this comment. up to you)
It looks relevant to both sites, so I'd probably keep it. It adds some noise, but it's better than having a chance of confusing someone by not including it.


Repository:
  rC Clang

https://reviews.llvm.org/D44439