[PATCH] D37182: [libcxx] Special visibility macros for the experimental library

Saleem Abdulrasool via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 31 22:55:38 PDT 2017


compnerd added a comment.

I think that splitting this up in a series of patches would be much better.  The first patch should be to do the entirely mechanical change of the visibility attribute.  It is a separate library and needs its own visibility attribute.  That would significantly slim down this subsequent change.  A follow-up change could fix the build, and a third change adds the test harness/support the tests.


https://reviews.llvm.org/D37182





More information about the cfe-commits mailing list