rriddle accepted this revision. rriddle added a comment. This revision is now accepted and ready to land.
LGTM, and makes sense to me given that downstream users often want a build of LLVM with `CMAKE_CXX_VISIBILITY_PRESET=hidden`, but lldb should still be able to build/link/work in the presence of that (the LLDB driver and tools currently fail to link if LLVM is built `CMAKE_CXX_VISIBILITY_PRESET=hidden`). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147453/new/ https://reviews.llvm.org/D147453 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
