[cfe-dev] ASTMatchers.h and its internal linkage variable definitions
Manuel Klimek via cfe-dev
cfe-dev at lists.llvm.org
Thu Nov 9 00:30:21 PST 2017
On Thu, Nov 9, 2017 at 12:43 AM David Blaikie <dblaikie at gmail.com> wrote:
> Hey Manuel,
> In an effort to apply Modular Code Generation to Clang internally, I came
> across the fact that this header contains a bunch of internal linkage
> variables. (now, maybe modular code generation should be compatible with
> internal linkage namespace-scoped variables... I'm looking into that)
> These variables technically create ODR violations any time they're
> ODR-used in another inline function used in more than one translation unit.
> Is there a good way we could fix this header so it doesn't create ODR
> violations (hopefully by not containing internal linkage entities)? I mean
> one simple solution is to declare all these things in a header and define
> them out of line. Since they have no actual state, that's probably pretty
> low-cost except for a lot of duplication. I'm happy to use a .def/.inc file
> to help remove a bunch of that duplication, if useful.
I remember having that argument a long time ago with somebody (perhaps Sam
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-dev