libstdc++: Fix outdated comment about std::string instantiations (PR 94854)
authorJonathan Wakely <jwakely@redhat.com>
Wed, 29 Apr 2020 17:57:34 +0000 (18:57 +0100)
committerJonathan Wakely <jwakely@redhat.com>
Wed, 29 Apr 2020 18:00:58 +0000 (19:00 +0100)
PR libstdc++/94854
* include/bits/basic_string.tcc: Update comment about explicit
instantiations.

libstdc++-v3/ChangeLog
libstdc++-v3/include/bits/basic_string.tcc

index f8cf71e057f19043958063a090b114fd4d971779..2032e90abebec9f5216369a08e56ad3adb85b982 100644 (file)
@@ -1,3 +1,9 @@
+2020-04-29  Jonathan Wakely  <jwakely@redhat.com>
+
+       PR libstdc++/94854
+       * include/bits/basic_string.tcc: Update comment about explicit
+       instantiations.
+
 2020-04-28  Jonathan Wakely  <jwakely@redhat.com>
 
        PR libstdc++/91480
index e78bb2c14b322bffaac4866f78de9d0bbd3695e3..e370f439390f94903988478a3c9bf36c169d94f5 100644 (file)
@@ -1609,11 +1609,11 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION
   // Inhibit implicit instantiations for required instantiations,
   // which are defined via explicit instantiations elsewhere.
 #if _GLIBCXX_EXTERN_TEMPLATE
-  // The explicit instantiations definitions in src/c++11/string-inst.cc
-  // are compiled as C++14, so the new C++17 members aren't instantiated.
-  // Until those definitions are compiled as C++17 suppress the declaration,
-  // so C++17 code will implicitly instantiate std::string and std::wstring
-  // as needed.
+  // The explicit instantiation definitions in src/c++11/string-inst.cc and
+  // src/c++17/string-inst.cc only instantiate the members required for C++17
+  // and earlier standards (so not C++20's starts_with and ends_with).
+  // Suppress the explicit instantiation declarations for C++20, so C++20
+  // code will implicitly instantiate std::string and std::wstring as needed.
 # if __cplusplus <= 201703L && _GLIBCXX_EXTERN_TEMPLATE > 0
   extern template class basic_string<char>;
 # elif ! _GLIBCXX_USE_CXX11_ABI