[PATCH] D52405: [libc++] Use exclude_from_explicit_instantiation instead of always_inline

[Richard Smith - zygoloid via Phabricator]

rsmith added a comment.

In https://reviews.llvm.org/D52405#1254544, @ldionne wrote:

> In https://reviews.llvm.org/D52405#1254526, @rsmith wrote:
>
> > Makes sense to me. Is there some way you can write a test for this? (Maybe applying `nm` to a linked program?)
>
>
> What I had in mind is that we should have a build bot that runs the test suite with/without `_LIBCPP_HIDE_FROM_ABI_PER_TU`. If all the current libc++ tests link, this seems to give us the most confidence that nothing is broken. Did you have something more specific in mind that you're concerned about? If so, I'm happy to write a test for it, I'd just like to understand what that is.


I'm happy with any testing that works :)


Repository:
  rCXX libc++

https://reviews.llvm.org/D52405