hiraditya created this revision. hiraditya added reviewers: mclow.lists, EricWF. hiraditya added subscribers: laxmansole, sebpop, cfe-commits.
Author: laxmansole Original Differential Revision: https://reviews.llvm.org/D22834 Posting the patch again as the bug https://llvm.org/bugs/show_bug.cgi?id=30341 is fixed. Currently basic_string's destructor is not getting inlined. So adding 'inline' attribute to ~basic_string(). Worked in collaboration with Aditya Kumar. git-svn-id: https://llvm.org/svn/llvm-project/libcxx/trunk@280944 91177308-0d34-0410-b5e6-96231b3b80d8 https://reviews.llvm.org/D25624 Files: include/string Index: include/string =================================================================== --- include/string +++ include/string @@ -1834,6 +1834,7 @@ #endif // _LIBCPP_HAS_NO_GENERALIZED_INITIALIZERS template <class _CharT, class _Traits, class _Allocator> +inline _LIBCPP_INLINE_VISIBILITY basic_string<_CharT, _Traits, _Allocator>::~basic_string() { #if _LIBCPP_DEBUG_LEVEL >= 2
Index: include/string =================================================================== --- include/string +++ include/string @@ -1834,6 +1834,7 @@ #endif // _LIBCPP_HAS_NO_GENERALIZED_INITIALIZERS template <class _CharT, class _Traits, class _Allocator> +inline _LIBCPP_INLINE_VISIBILITY basic_string<_CharT, _Traits, _Allocator>::~basic_string() { #if _LIBCPP_DEBUG_LEVEL >= 2
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits