[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.
This commit is contained in:
parent
21edd84161
commit
8be49eb927
2 changed files with 11 additions and 0 deletions
|
@ -36,6 +36,11 @@ int
|
|||
main()
|
||||
{
|
||||
using namespace std;
|
||||
// Ensure debug info for std::string is issued in the local
|
||||
// translation unit, so that GDB won't pick up any alternate
|
||||
// std::string notion that might be present in libstdc++.so.
|
||||
string bah = "hi";
|
||||
(void)bah;
|
||||
unique_ptr<vector<unique_ptr<vector<int>*>>> p1;
|
||||
unique_ptr<vector<unique_ptr<set<int>*>>[]> p2;
|
||||
unique_ptr<set<unique_ptr<vector<int>*>>[10]> p3;
|
||||
|
|
|
@ -32,6 +32,12 @@ using std::experimental::string_view;
|
|||
int
|
||||
main()
|
||||
{
|
||||
// Ensure debug info for std::string is issued in the local
|
||||
// translation unit, so that GDB won't pick up any alternate
|
||||
// std::string notion that might be present in libstdc++.so.
|
||||
std::string bah = "hi";
|
||||
(void)bah;
|
||||
|
||||
string_view str = "string";
|
||||
// { dg-final { note-test str "\"string\"" } }
|
||||
|
||||
|
|
Loading…
Add table
Reference in a new issue