Jan =?utf-8?q?Kokemüller?= <[email protected]>,
Jan =?utf-8?q?Kokemüller?= <[email protected]>,
Jan =?utf-8?q?Kokemüller?= <[email protected]>,
Jan =?utf-8?q?Kokemüller?= <[email protected]>,
Jan =?utf-8?q?Kokemüller?= <[email protected]>,
Jan =?utf-8?q?Kokemüller?= <[email protected]>,
Jan =?utf-8?q?Kokemüller?= <[email protected]>,
Jan =?utf-8?q?Kokemüller?= <[email protected]>,
Jan =?utf-8?q?Kokemüller?= <[email protected]>,
Jan =?utf-8?q?Kokemüller?= <[email protected]>,
Jan =?utf-8?q?Kokemüller?= <[email protected]>,
Jan =?utf-8?q?Kokemüller?= <[email protected]>,
Jan =?utf-8?q?Kokemüller?= <[email protected]>,
Jan =?utf-8?q?Kokemüller?= <[email protected]>,
Jan =?utf-8?q?Kokemüller?= <[email protected]>,
Jan =?utf-8?q?Kokemüller?= <[email protected]>,
Jan =?utf-8?q?Kokemüller?= <[email protected]>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/[email protected]>
================
@@ -287,32 +273,33 @@ Sema::getCurrentMangleNumberContext(const DeclContext
*DC) {
DataMember,
InlineVariable,
TemplatedVariable,
+ ExternallyVisibleVariableInModulePurview,
Concept,
- NonInlineInModulePurview
} Kind = Normal;
bool IsInNonspecializedTemplate =
inTemplateInstantiation() || CurContext->isDependentContext();
+ // Checks if a VarDecl or FunctionDecl is from a module purview and
externally
+ // visible. These Decls should be treated as "inline" for the purpose of
+ // mangling in the code below.
+ //
+ // See discussion in https://github.com/itanium-cxx-abi/cxx-abi/issues/186
+ //
+ // zygoloid:
+ // Yeah, I think the only cases left where lambdas don't need a
+ // mangling are when they have (effectively) internal linkage or
+ // appear in a non-inline function in a non-module translation unit.
----------------
cor3ntin wrote:
```suggestion
```
https://github.com/llvm/llvm-project/pull/179035
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
