[PATCH] D27068: Improve string::find

Eric Fiselier via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 30 02:26:05 PST 2016


EricWF accepted this revision.
EricWF added a comment.
This revision is now accepted and ready to land.

Holy crap those improvements are impressive.

This LGTM. I'm assuming @mclow.lists has nothing left to say about this.



================
Comment at: libcxx/include/__string:542
+template <class _CharT, class _Traits>
+_LIBCPP_CONSTEXPR_AFTER_CXX11
+const _CharT *
----------------
Please add `inline` to the template so `-fvisibility-inlines-hidden` works with it.


https://reviews.llvm.org/D27068





More information about the cfe-commits mailing list