[PATCH] D101972: Do not set CMAKE_CXX_VISIBILITY_PRESET to hidden

Tom Stellard via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 6 07:07:29 PDT 2021


tstellar added a comment.

Adding @rnk, who reviewed the original patch for awareness...

It's unfortunate, but I'm fine with this change.  We could workaround the specific issue with llvm::Any another way, but there is nothing in place that would prevent someone else from adding a new feature that would hit the same issue again.    It seems like disabling this completely is the correct approach, and we will need to come up with a different strategy to limit the number of exported symbols.


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

https://reviews.llvm.org/D101972



More information about the llvm-commits mailing list