[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