[PATCH] D118018: [RFC] [C+++20] [Modules] Mangling lambda in modules

2022-01-24 Thread Nathan Sidwell via Phabricator via cfe-commits
urnathan added a comment. sure, please reassign them Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118018/new/ https://reviews.llvm.org/D118018 ___ cfe-commits mailing list cfe-commits@lists.llvm.org htt

[PATCH] D118018: [RFC] [C+++20] [Modules] Mangling lambda in modules

2022-01-24 Thread Chuanqi Xu via Phabricator via cfe-commits
ChuanqiXu added a comment. In D118018#3265810 , @urnathan wrote: > I don't think this is wanted. there are two cases: > a) lambda is attached to some ODR-visible entitity. Something like (in > module-purview at namespace-scope) '[maybe-export] auto va

[PATCH] D118018: [RFC] [C+++20] [Modules] Mangling lambda in modules

2022-01-24 Thread Nathan Sidwell via Phabricator via cfe-commits
urnathan added a comment. I don't think this is wanted. there are two cases: a) lambda is attached to some ODR-visible entitity. Something like (in module-purview at namespace-scope) '[maybe-export] auto var = []{};'. Here the lambda acquires 'var' as its context. b) lambda is not attached l

[PATCH] D118018: [RFC] [C+++20] [Modules] Mangling lambda in modules

2022-01-24 Thread Chuanqi Xu via Phabricator via cfe-commits
ChuanqiXu created this revision. ChuanqiXu added reviewers: urnathan, aaron.ballman, dblaikie, rsmith, erichkeane, efriedma, gbenyei. ChuanqiXu added a project: clang. ChuanqiXu requested review of this revision. Herald added a subscriber: cfe-commits. This tries to solve https://github.com/llvm/