[PATCH] D39627: Fix vtable not receiving hidden visibility when using push(visibility)

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 6 14:35:37 PST 2017

efriedma added subscribers: cfe-commits, efriedma.
efriedma added a comment.

Adding cfe-commits.  (In the future, please make sure to CC that list instead of llvm-commits for clang changes.)

> I'm missing some background lingo here. What is the meaning of "a declaration" and "a definition" here?

In most cases, we only emit the vtable for a class in one object file.  Informally, we can call that the "definition" of the vtable, as an analogy to variables and functions.  See also http://itanium-cxx-abi.github.io/cxx-abi/abi.html#vague-vtable .



More information about the cfe-commits mailing list