[PATCH] D25593: [libcxx] Build with -fvisibility-inlines-hidden -- Remove 20 inline definitions from the dylib

Ed Maste via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 14 06:54:19 PDT 2016


emaste added inline comments.


================
Comment at: lib/abi/CHANGELOG.TXT:22
+  Although this change removes symbols, it should still be non-ABI breaking
+  since all of the definitions removed are inline functions.
+
----------------
Should we also include in this comment the further justification from your Phabricator description above (about having own definitions)?


https://reviews.llvm.org/D25593





More information about the cfe-commits mailing list