]> git.ipfire.org Git - thirdparty/gcc.git/commit
[libstdc++] [prettyprint] add local std::string use to more tests
authorAlexandre Oliva <oliva@adacore.com>
Fri, 3 Mar 2023 18:59:33 +0000 (15:59 -0300)
committerAlexandre Oliva <oliva@gnu.org>
Fri, 3 Mar 2023 19:06:46 +0000 (16:06 -0300)
commit8be49eb927e1e65a715bc6eb73b9ce271c8b24c2
treedd2f3e9629770df50a61081612fab7debd195279
parent21edd841611a97442a6b95e8ec7e91ff8fd3a451
[libstdc++] [prettyprint] add local std::string use to more tests

We're getting regressions after ugprading to GDB 13 in a few of the
libstdc++ pretty-printing tests.  It might seem like a GDB regression,
but that update has a new symbol reader, and Tom Tromey says it
exposes a latent problem in the pretty-printer, namely, when a name is
not defined locally in the local translation unit, GDB will look for
it in other units, and if multiple different definitions are
available, there are no guarantees as to which one it will find.

Since libstdc++ contains units in which std::string maps to different
types, in order to ensure the tests get the std::string definition we
expect, we need std::string to be present in the debug information for
the test itself, as we already do in other tests.

for  libstdc++-v3/ChangeLog

* testsuite/libstdc++-prettyprinters/80276.cc: Add
std::string to debug info.
* testsuite/libstdc++-prettyprinters/libfundts.cc: Likewise.
libstdc++-v3/testsuite/libstdc++-prettyprinters/80276.cc
libstdc++-v3/testsuite/libstdc++-prettyprinters/libfundts.cc