[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