[PATCH] D54439: CMake: Make most target symbols hidden by default

Mon Nov 11 07:09:21 PST 2019

ldionne added a comment.

In D54439#1722049 <https://reviews.llvm.org/D54439#1722049>, @tstellar wrote:

> Rebase on trunk.
>
> The warnings are still present on OS X.  Here is a build log:
>  https://github.com/tstellar/llvm-project/pull/11/checks?check_run_id=275366813

@tstellar Can you add `_LIBCPP_INLINE_VISIBILITY` to `__call_once_proxy` and see whether that fixes the warning? I believe that is the correct fix, now that we don't use `always_inline` anymore to implement `_LIBCPP_INLINE_VISIBILITY`.

I've added `_LIBCPP_INLINE_VISIBILITY` to `__call_once_proxy` locally and everything works, but I don't have your specific patch to test locally.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D54439