You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
119 lines
5.7 KiB
119 lines
5.7 KiB
5 months ago
|
From 59388ee3dac5c737086ed1f83f552a44481a2213 Mon Sep 17 00:00:00 2001
|
||
|
From: David Malcolm <dmalcolm@redhat.com>
|
||
|
Date: Tue, 31 Aug 2021 16:08:57 -0400
|
||
|
Subject: [PATCH 06/17] Don't assume has_facet<codecvt_c16> when run against
|
||
|
system libstdc++
|
||
|
|
||
|
Upstream commit 2019-02-19:
|
||
|
P0482R5 char8_t: Standard library support
|
||
|
https://gcc.gnu.org/git/?p=gcc.git;a=commitdiff;h=c124af936b6b225eb548ccdd7f01400511d784dc
|
||
|
added new locale facets for char8_t.
|
||
|
|
||
|
has_facet<codecvt_c16> returns false, as it is using the std::locale the system libstdc++:
|
||
|
|
||
|
$ eu-readelf -s char16_t-char8_t.exe |c++filt |grep UNDEF|grep locale
|
||
|
3: 0000000000000000 0 FUNC GLOBAL DEFAULT UNDEF std::locale::classic()@GLIBCXX_3.4 (2)
|
||
|
8: 0000000000000000 0 FUNC GLOBAL DEFAULT UNDEF std::locale::locale(std::locale const&)@GLIBCXX_3.4 (2)
|
||
|
13: 0000000000000000 0 FUNC GLOBAL DEFAULT UNDEF std::locale::id::_M_id() const@GLIBCXX_3.4 (2)
|
||
|
22: 0000000000000000 0 FUNC GLOBAL DEFAULT UNDEF std::locale::~locale()@GLIBCXX_3.4 (2)
|
||
|
64: 0000000000000000 0 FUNC GLOBAL DEFAULT UNDEF std::locale::classic()@@GLIBCXX_3.4
|
||
|
76: 0000000000000000 0 FUNC GLOBAL DEFAULT UNDEF std::locale::locale(std::locale const&)@@GLIBCXX_3.4
|
||
|
89: 0000000000000000 0 FUNC GLOBAL DEFAULT UNDEF std::locale::id::_M_id() const@@GLIBCXX_3.4
|
||
|
108: 0000000000000000 0 FUNC GLOBAL DEFAULT UNDEF std::locale::~locale()@@GLIBCXX_3.4
|
||
|
|
||
|
Hence this patch tweaks tests so that they bail out if run against an older std::locale.
|
||
|
---
|
||
|
libstdc++-v3/testsuite/22_locale/codecvt/char16_t-char8_t.cc | 3 ++-
|
||
|
libstdc++-v3/testsuite/22_locale/codecvt/char16_t.cc | 3 ++-
|
||
|
libstdc++-v3/testsuite/22_locale/codecvt/char32_t-char8_t.cc | 3 ++-
|
||
|
libstdc++-v3/testsuite/22_locale/codecvt/char32_t.cc | 3 ++-
|
||
|
libstdc++-v3/testsuite/22_locale/codecvt/utf8-char8_t.cc | 2 ++
|
||
|
libstdc++-v3/testsuite/22_locale/codecvt/utf8.cc | 2 ++
|
||
|
6 files changed, 12 insertions(+), 4 deletions(-)
|
||
|
|
||
|
diff --git a/libstdc++-v3/testsuite/22_locale/codecvt/char16_t-char8_t.cc b/libstdc++-v3/testsuite/22_locale/codecvt/char16_t-char8_t.cc
|
||
|
index 71a23bef4..d4aa519a2 100644
|
||
|
--- a/libstdc++-v3/testsuite/22_locale/codecvt/char16_t-char8_t.cc
|
||
|
+++ b/libstdc++-v3/testsuite/22_locale/codecvt/char16_t-char8_t.cc
|
||
|
@@ -31,7 +31,8 @@ test01()
|
||
|
using namespace std;
|
||
|
typedef codecvt<char16_t, char8_t, mbstate_t> codecvt_c16;
|
||
|
locale loc_c = locale::classic();
|
||
|
- VERIFY(has_facet<codecvt_c16>(loc_c));
|
||
|
+ if (!has_facet<codecvt_c16>(loc_c))
|
||
|
+ return;
|
||
|
const codecvt_c16* const cvt = &use_facet<codecvt_c16>(loc_c);
|
||
|
|
||
|
VERIFY(!cvt->always_noconv());
|
||
|
diff --git a/libstdc++-v3/testsuite/22_locale/codecvt/char16_t.cc b/libstdc++-v3/testsuite/22_locale/codecvt/char16_t.cc
|
||
|
index 7b5ce126d..05c77b2b1 100644
|
||
|
--- a/libstdc++-v3/testsuite/22_locale/codecvt/char16_t.cc
|
||
|
+++ b/libstdc++-v3/testsuite/22_locale/codecvt/char16_t.cc
|
||
|
@@ -29,7 +29,8 @@ test01()
|
||
|
using namespace std;
|
||
|
typedef codecvt<char16_t, char, mbstate_t> codecvt_c16;
|
||
|
locale loc_c = locale::classic();
|
||
|
- VERIFY(has_facet<codecvt_c16>(loc_c));
|
||
|
+ if (!has_facet<codecvt_c16>(loc_c))
|
||
|
+ return;
|
||
|
const codecvt_c16* const cvt = &use_facet<codecvt_c16>(loc_c);
|
||
|
|
||
|
VERIFY(!cvt->always_noconv());
|
||
|
diff --git a/libstdc++-v3/testsuite/22_locale/codecvt/char32_t-char8_t.cc b/libstdc++-v3/testsuite/22_locale/codecvt/char32_t-char8_t.cc
|
||
|
index f30ae22e9..281969e9b 100644
|
||
|
--- a/libstdc++-v3/testsuite/22_locale/codecvt/char32_t-char8_t.cc
|
||
|
+++ b/libstdc++-v3/testsuite/22_locale/codecvt/char32_t-char8_t.cc
|
||
|
@@ -33,7 +33,8 @@ test01()
|
||
|
using namespace std;
|
||
|
typedef codecvt<char32_t, char8_t, mbstate_t> codecvt_c32;
|
||
|
locale loc_c = locale::classic();
|
||
|
- VERIFY(has_facet<codecvt_c32>(loc_c));
|
||
|
+ if (!has_facet<codecvt_c32>(loc_c))
|
||
|
+ return;
|
||
|
const codecvt_c32* const cvt = &use_facet<codecvt_c32>(loc_c);
|
||
|
|
||
|
VERIFY(!cvt->always_noconv());
|
||
|
diff --git a/libstdc++-v3/testsuite/22_locale/codecvt/char32_t.cc b/libstdc++-v3/testsuite/22_locale/codecvt/char32_t.cc
|
||
|
index 558ba9145..82980b702 100644
|
||
|
--- a/libstdc++-v3/testsuite/22_locale/codecvt/char32_t.cc
|
||
|
+++ b/libstdc++-v3/testsuite/22_locale/codecvt/char32_t.cc
|
||
|
@@ -31,7 +31,8 @@ test01()
|
||
|
using namespace std;
|
||
|
typedef codecvt<char32_t, char, mbstate_t> codecvt_c32;
|
||
|
locale loc_c = locale::classic();
|
||
|
- VERIFY(has_facet<codecvt_c32>(loc_c));
|
||
|
+ if (!has_facet<codecvt_c32>(loc_c))
|
||
|
+ return;
|
||
|
const codecvt_c32* const cvt = &use_facet<codecvt_c32>(loc_c);
|
||
|
|
||
|
VERIFY(!cvt->always_noconv());
|
||
|
diff --git a/libstdc++-v3/testsuite/22_locale/codecvt/utf8-char8_t.cc b/libstdc++-v3/testsuite/22_locale/codecvt/utf8-char8_t.cc
|
||
|
index 5eab05ba2..d6f663ce6 100644
|
||
|
--- a/libstdc++-v3/testsuite/22_locale/codecvt/utf8-char8_t.cc
|
||
|
+++ b/libstdc++-v3/testsuite/22_locale/codecvt/utf8-char8_t.cc
|
||
|
@@ -34,6 +34,8 @@ void test(const C* from)
|
||
|
std::mbstate_t state{};
|
||
|
char8_t buf[16] = { };
|
||
|
using test_type = std::codecvt<C, char8_t, std::mbstate_t>;
|
||
|
+ if (!std::has_facet<test_type>(std::locale::classic()))
|
||
|
+ return;
|
||
|
const test_type& cvt = std::use_facet<test_type>(std::locale::classic());
|
||
|
auto from_end = from + len;
|
||
|
auto from_next = from;
|
||
|
diff --git a/libstdc++-v3/testsuite/22_locale/codecvt/utf8.cc b/libstdc++-v3/testsuite/22_locale/codecvt/utf8.cc
|
||
|
index b1b3cff31..167bc0ca3 100644
|
||
|
--- a/libstdc++-v3/testsuite/22_locale/codecvt/utf8.cc
|
||
|
+++ b/libstdc++-v3/testsuite/22_locale/codecvt/utf8.cc
|
||
|
@@ -32,6 +32,8 @@ void test(const C* from)
|
||
|
std::mbstate_t state{};
|
||
|
char buf[16] = { };
|
||
|
using test_type = std::codecvt<C, char, std::mbstate_t>;
|
||
|
+ if (!std::has_facet<test_type>(std::locale::classic()))
|
||
|
+ return;
|
||
|
const test_type& cvt = std::use_facet<test_type>(std::locale::classic());
|
||
|
auto from_end = from + len;
|
||
|
auto from_next = from;
|
||
|
--
|
||
|
2.31.1
|
||
|
|