Re-enable libc++ gdb-pretty-printer tests for all versions of clang.

[saugustine]

Summary

The bootstrap tests were failing because the environment has a version
of gdb too old to read dwarf5 as generated by clang. The problem isn't
just pretty printer tests--this version of gdb will fail to read any
program compiled with those versions of clang.

More discussion here:

https://discourse.llvm.org/t/gdb-10-1-cant-read-clangs-dwarf-v5/6035/11

[Mordante]

Note the pre-commit CI uses Ubuntu 20.04 which contains gdb 9.1. Ubuntu 22.04 has just been released, @ldionne how do you feel about moving the CI to Ubuntu 22.04?

Alternatively we can try to add -gdwarf-4 to the RUN line.

[ldionne]

In D124569#3479235, @Mordante wrote:

Note the pre-commit CI uses Ubuntu 20.04 which contains gdb 9.1. Ubuntu 22.04 has just been released, @ldionne how do you feel about moving the CI to Ubuntu 22.04?

Alternatively we can try to add -gdwarf-4 to the RUN line.

I'm fine with moving to Ubuntu 22.04! As you know, however, platform updates are not always super smooth for the CI, so I think it may be good to do it over the week-end.

[ldionne]

This LGTM once we've moved to Ubuntu 22.04. Please wait for that and then rebase to you can get a clean CI run.

[ldionne]

We have now moved to Ubuntu 22.04, let's rebase this and give it a shot. We've been seeing issues with these tests so it might help.

[saugustine]

I can't manage to get the buildbot environment working to test this. It consistently fails with

clang: error: unable to execute command: posix_spawn failed: Operation not permitted

Not sure the best thing to do here. I think it should work fine, but am stumped on how to test it.

[Mordante]

In D124569#3578931, @saugustine wrote:

I can't manage to get the buildbot environment working to test this. It consistently fails with

clang: error: unable to execute command: posix_spawn failed: Operation not permitted

Not sure the best thing to do here. I think it should work fine, but am stumped on how to test it.

Can you rebase the patch so we can see the exact errors in our CI.
Note there are probably some merge conflicts since I added more builds to the exception list.

[saugustine]

Rebased changes

[Mordante]

Thanks! I see different errors now. I'll see what I can do about it. At the moment our CI is rather unstable so I'll look at it when that's fixed.

libcxx/test/libcxx/gdb/gdb_pretty_printer_test.sh.cpp

15

This hunk can probably also be removed. But I'll investigate the build issue first.

[dblaikie]

@Mordante ping on this - be nice to get this back up and running since the pretty printers have regressed recently (string_view regressed/broke due to a member rename from data to data_)

[saugustine]

Circling back to this.

Reenable libc++ gdb-pretty-printer tests
_and_ update the pretty printer for bitrot.

Will push this shortly as we had already agree it should be done.

[saugustine]

Committed as e4d5daaf91f6719626fa36c75e553e91778b94a9

[daltenty]

Seems like this is breaking in the libc++ CI: https://buildkite.com/llvm-project/libcxx-ci/builds/14027

[dblaikie]

In D124569#3852532, @daltenty wrote:

Seems like this is breaking in the libc++ CI: https://buildkite.com/llvm-project/libcxx-ci/builds/14027

looks like this came good one way or another? (at least I see some recent green runs)

libcxx/test/libcxx/gdb/gdb_pretty_printer_test.sh.cpp

 // UNSUPPORTED: no-localization
 // UNSUPPORTED: c++03
 
-// TODO: Investigate this failure, which happens only with the Bootstrapping build.
-// UNSUPPORTED: clang-13, clang-14, clang-15
-
 // TODO: Investigate this failure on GCC 12 (in Ubuntu Jammy)
 // UNSUPPORTED: gcc-12