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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits