[PATCH] D98068: Exampt asserts for recursive lambdas about LocalInstantiationScope

2021-06-14 Thread Yaxun Liu via Phabricator via cfe-commits
yaxunl updated this revision to Diff 351990. yaxunl edited the summary of this revision. yaxunl added a comment. Revised by Richard's comments. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98068/new/ https://reviews.llvm.org/D98068 Files: clang/lib/Sema/SemaTemplateInstantiateDecl.c

[PATCH] D98068: Exampt asserts for recursive lambdas about LocalInstantiationScope

2021-06-14 Thread Richard Smith - zygoloid via Phabricator via cfe-commits
rsmith added a comment. I think this is the wrong solution; I think the assertion is correct and is observing a problem elsewhere. Imagine if we had something like: template void f() { auto a = [] (auto recurse, auto x) { auto b = [] (auto) { refer to local decl in a

[PATCH] D98068: Exampt asserts for recursive lambdas about LocalInstantiationScope

2021-06-14 Thread Yaxun Liu via Phabricator via cfe-commits
yaxunl updated this revision to Diff 351934. yaxunl retitled this revision from "Remove asserts for LocalInstantiationScope" to "Exampt asserts for recursive lambdas about LocalInstantiationScope". yaxunl edited the summary of this revision. yaxunl added a comment. re-enable the assert but with e