[PATCH] D101972: For visibility of llvm::Any to external
Reid Kleckner via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue May 11 12:25:44 PDT 2021
rnk added a comment.
My first thought was to put it on the TypeId class, like so:
` template <typename T> struct LLVM_EXTERNAL_VISIBILITY TypeId {`
I don't think the `id()` method visibility matters, and fewer dynamic symbols is usually better.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D101972/new/
https://reviews.llvm.org/D101972
More information about the llvm-commits
mailing list