[PATCH] D108690: Explicit approach to controlling symbol visibility

serge via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 30 07:28:38 PDT 2021


serge-sans-paille added a comment.

In D108690#2968897 <https://reviews.llvm.org/D108690#2968897>, @jrtc27 wrote:

> Is there a strategy for downstreams so that we don't have a weird mishmash of symbol visibilities, where files we've added have the old rules applied but files present upstream have the new rules?

I'm going to drop that patch, it's too intrusive. the alternative approach is to flag some classes used as parameter of `llvm::Any` as externally visible, I'm going to follow that way.


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

https://reviews.llvm.org/D108690



More information about the llvm-commits mailing list