[libcxx-commits] [PATCH] D152378: [libc++][filesystem] Use _LIBCPP_HIDE_FROM_ABI in common headers

[Louis Dionne via Phabricator via libcxx-commits]

ldionne added a comment.

Actually I just thought about this and if users are linking against static `libc++.a`, then they could potentially end up with symbols with the same name from different versions of libc++, which would be an ODR violation. In other words, we do need the same precautions in our own built library as we have in our headers due to static archives. So I believe `_LIBCPP_HIDE_FROM_ABI` is necessary if we drop the anonymous namespace. Do you agree?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152378