libstdc++: Remove <memory_resource> dependency from <regex> [PR 92546]
authorJonathan Wakely <jwakely@redhat.com>
Fri, 20 Nov 2020 11:30:33 +0000 (11:30 +0000)
committerJonathan Wakely <jwakely@redhat.com>
Fri, 20 Nov 2020 13:06:48 +0000 (13:06 +0000)
commit640ebeb336050887cb57417b7568279c588088f0
tree8cc829385b7e1d8d640ecc193a7a09b9fb4552fc
parent4405edb496483c5e99adb311cc694b0b063eb358
libstdc++: Remove <memory_resource> dependency from <regex> [PR 92546]

Unlike the other headers that declare alias templates in namespace pmr,
<regex> includes <memory_resource>. That was done because the
pmr::string::const_iterator typedef requires pmr::string to be complete,
which requires pmr::polymorphic_allocator<char> to be complete.

By using __normal_iterator<const char*, pmr::string> instead of the
const_iterator typedef we can avoid the completeness requirement.

This makes <regex> smaller, by not requiring <memory_resource> and its
<shared_mutex> dependency, which depends on <chrono>.  Backporting this
will also help with PR 97876, where <stop_token> ends up being needed by
<regex> via <memory_resource>.

libstdc++-v3/ChangeLog:

PR libstdc++/92546
* include/std/regex (pmr::smatch, pmr::wsmatch): Declare using
underlying __normal_iterator type, not nested typedef
basic_string::const_iterator.
libstdc++-v3/include/std/regex