[PATCH] D25624: Added 'inline' attribute to basic_string's destructor

Marshall Clow via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 19 11:12:43 PDT 2016


mclow.lists added a comment.

I don't have a problem with this being marked as inline, as long as it doesn't disappear out of the dylib.

There *has* to be a version of `basic_string<char, char_traits<char>, Allocator<char>>::~basic_string` in the dylib - existing applications depend upon it. (same for `wchar_t`).


https://reviews.llvm.org/D25624





More information about the cfe-commits mailing list