[PATCH] D101972: For visibility of llvm::Any to external

serge via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 11 11:53:33 PDT 2021


serge-sans-paille added a comment.

@rnk setting the whole class as external is the less intrusive patch and I've tested it successfully. Maybe it's enough to set the default visibility to `id()` and `template <typename T> const char Any::TypeId<T>::Id = 0` ?


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

https://reviews.llvm.org/D101972



More information about the llvm-commits mailing list