[PATCH] D138247: PR58819: Correct linkage and mangling of lambdas in inline static member initializers

Thu Mar 23 10:40:02 PDT 2023

efriedma added a comment.

The relevant text of the current Itanium ABI (which was updated in https://github.com/itanium-cxx-abi/cxx-abi/commit/d8e9d102c83f177970f0db6cc8bee170f2779bc1)

> In the following contexts, however, the one-definition rule requires closure types in different translation units to "correspond":
>
> - default arguments appearing in class definitions
> - default member initializers
> - the bodies of inline or templated functions
> - the initializers of inline or templated variables

Could you update the references to the ABI document to use the new text?

Given the new rules, I think ContextKind::StaticDataMember shouldn't exist?  It's not one of the given categories; should be subsumed by the existing categories.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D138247/new/

https://reviews.llvm.org/D138247