public inbox for gcc-cvs@sourceware.org
help / color / mirror / Atom feed
* [gcc r11-9313] libstdc++: Fix vector<bool> printer tests to work in debug mode
@ 2021-11-25 19:05 Jonathan Wakely
0 siblings, 0 replies; only message in thread
From: Jonathan Wakely @ 2021-11-25 19:05 UTC (permalink / raw)
To: gcc-cvs, libstdc++-cvs
https://gcc.gnu.org/g:88b9997203637f05a36144e8f51f7d5b2dba040a
commit r11-9313-g88b9997203637f05a36144e8f51f7d5b2dba040a
Author: Jonathan Wakely <jwakely@redhat.com>
Date: Tue Aug 17 15:06:22 2021 +0100
libstdc++: Fix vector<bool> printer tests to work in debug mode
This fixes a compilation error in debug mode, due to std::_Bit_reference
not being defined, because it's in namespace std::__cxx1998 instead. We
can refer to it as vector<bool>::reference instead, which always works.
That fixes some compilation errors in debug mode, but the tests fail at
run-time instead because the printers for vector<bool> helpers are only
registered for the std namespace, not std::__cxx1998. That is fixed by
using add_container to register the printers instead of add_version, as
the former registers them in the std and std::__cxx1998 namespaces.
Signed-off-by: Jonathan Wakely <jwakely@redhat.com>
libstdc++-v3/ChangeLog:
* python/libstdcxx/v6/printers.py (StdBitReferencePrinter): Use
'std::vector<bool>::reference' as type name, not _Bit_reference.
(build_libstdcxx_dictionary): Register printers for vector<bool>
types in debug mode too.
* testsuite/libstdc++-prettyprinters/simple.cc: Adjust expected
output for invalid _Bit_reference. Use vector<bool>::reference
instead of _Bit_reference.
* testsuite/libstdc++-prettyprinters/simple11.cc: Likewise.
(cherry picked from commit c883d1dcdea8820015ee7ffdc68c03cf3b8180bf)
Diff:
---
libstdc++-v3/python/libstdcxx/v6/printers.py | 10 +++++-----
libstdc++-v3/testsuite/libstdc++-prettyprinters/simple.cc | 14 +++++++-------
.../testsuite/libstdc++-prettyprinters/simple11.cc | 14 +++++++-------
3 files changed, 19 insertions(+), 19 deletions(-)
diff --git a/libstdc++-v3/python/libstdcxx/v6/printers.py b/libstdc++-v3/python/libstdcxx/v6/printers.py
index 25c1d2b1206..c5072c52281 100644
--- a/libstdc++-v3/python/libstdcxx/v6/printers.py
+++ b/libstdc++-v3/python/libstdcxx/v6/printers.py
@@ -522,14 +522,14 @@ class StdBitIteratorPrinter:
return bool(self.val['_M_p'].dereference() & (1 << self.val['_M_offset']))
class StdBitReferencePrinter:
- "Print std::_Bit_reference"
+ "Print std::vector<bool>::reference"
def __init__(self, typename, val):
self.val = val
def to_string(self):
if not self.val['_M_p']:
- return 'invalid std::_Bit_reference'
+ return 'invalid std::vector<bool>::reference'
return bool(self.val['_M_p'].dereference() & (self.val['_M_mask']))
class StdTuplePrinter:
@@ -2083,11 +2083,11 @@ def build_libstdcxx_dictionary ():
StdDequeIteratorPrinter)
libstdcxx_printer.add_version('__gnu_cxx::', '__normal_iterator',
StdVectorIteratorPrinter)
- libstdcxx_printer.add_version('std::', '_Bit_iterator',
+ libstdcxx_printer.add_container('std::', '_Bit_iterator',
StdBitIteratorPrinter)
- libstdcxx_printer.add_version('std::', '_Bit_const_iterator',
+ libstdcxx_printer.add_container('std::', '_Bit_const_iterator',
StdBitIteratorPrinter)
- libstdcxx_printer.add_version('std::', '_Bit_reference',
+ libstdcxx_printer.add_container('std::', '_Bit_reference',
StdBitReferencePrinter)
libstdcxx_printer.add_version('__gnu_cxx::', '_Slist_iterator',
StdSlistIteratorPrinter)
diff --git a/libstdc++-v3/testsuite/libstdc++-prettyprinters/simple.cc b/libstdc++-v3/testsuite/libstdc++-prettyprinters/simple.cc
index c1fc4b1fe72..02f762efe28 100644
--- a/libstdc++-v3/testsuite/libstdc++-prettyprinters/simple.cc
+++ b/libstdc++-v3/testsuite/libstdc++-prettyprinters/simple.cc
@@ -144,19 +144,19 @@ main()
std::vector<bool>::iterator vbIt0;
// { dg-final { note-test vbIt0 {non-dereferenceable iterator for std::vector<bool>} } }
- std::_Bit_reference br = *vb.begin();
+ std::vector<bool>::reference br = *vb.begin();
// { dg-final { note-test br {true} } }
- std::_Bit_reference br2 = *vbIt2;
+ std::vector<bool>::reference br2 = *vbIt2;
// { dg-final { note-test br2 {true} } }
- std::_Bit_reference br3 = *vbIt3;
+ std::vector<bool>::reference br3 = *vbIt3;
// { dg-final { note-test br3 {false} } }
- std::_Bit_reference br4 = *vbIt4;
+ std::vector<bool>::reference br4 = *vbIt4;
// { dg-final { note-test br4 {false} } }
- std::_Bit_reference br5 = *vbIt5;
+ std::vector<bool>::reference br5 = *vbIt5;
// { dg-final { note-test br5 {true} } }
- std::_Bit_reference br0;
-// { dg-final { note-test br0 {invalid std::_Bit_reference} } }
+ std::vector<bool>::reference br0;
+// { dg-final { note-test br0 {invalid std::vector<bool>::reference} } }
__gnu_cxx::slist<int> sll;
sll.push_front(23);
diff --git a/libstdc++-v3/testsuite/libstdc++-prettyprinters/simple11.cc b/libstdc++-v3/testsuite/libstdc++-prettyprinters/simple11.cc
index d219e271ff3..bd6e026a329 100644
--- a/libstdc++-v3/testsuite/libstdc++-prettyprinters/simple11.cc
+++ b/libstdc++-v3/testsuite/libstdc++-prettyprinters/simple11.cc
@@ -137,19 +137,19 @@ main()
std::vector<bool>::iterator vbIt0;
// { dg-final { note-test vbIt0 {non-dereferenceable iterator for std::vector<bool>} } }
- std::_Bit_reference br = *vb.begin();
+ std::vector<bool>::reference br = *vb.begin();
// { dg-final { note-test br {true} } }
- std::_Bit_reference br2 = *vbIt2;
+ std::vector<bool>::reference br2 = *vbIt2;
// { dg-final { note-test br2 {true} } }
- std::_Bit_reference br3 = *vbIt3;
+ std::vector<bool>::reference br3 = *vbIt3;
// { dg-final { note-test br3 {false} } }
- std::_Bit_reference br4 = *vbIt4;
+ std::vector<bool>::reference br4 = *vbIt4;
// { dg-final { note-test br4 {false} } }
- std::_Bit_reference br5 = *vbIt5;
+ std::vector<bool>::reference br5 = *vbIt5;
// { dg-final { note-test br5 {true} } }
- std::_Bit_reference br0;
-// { dg-final { note-test br0 {invalid std::_Bit_reference} } }
+ std::vector<bool>::reference br0;
+// { dg-final { note-test br0 {invalid std::vector<bool>::reference} } }
__gnu_cxx::slist<int> sll;
sll.push_front(23);
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2021-11-25 19:05 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-25 19:05 [gcc r11-9313] libstdc++: Fix vector<bool> printer tests to work in debug mode Jonathan Wakely
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).