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

[Laxman Sole via cfe-commits]

laxmansole created this revision.
laxmansole added reviewers: mclow.lists, howard.hinnant.
laxmansole added subscribers: cfe-commits, sebpop, hiraditya, evandro, flyingforyou.


Currently basic_string's destructor is not getting inlined. So adding 'inline' attribute to ~basic_string().


Worked in collaboration with Aditya Kumar.

https://reviews.llvm.org/D22834

Files:
  libcxx/include/string

Index: libcxx/include/string
===================================================================
--- libcxx/include/string
+++ libcxx/include/string
@@ -2350,6 +2350,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


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D22834.65591.patch
Type: text/x-patch
Size: 410 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160726/2ecb1190/attachment.bin>