[libc++] Add UNSUPPORTED for clang-15

[haowei]

Summary

This started breaking in the CI because we bumped the Clang version to 15, which requires adjusting the markup in the test suite.

[philnik]

libcxx/test/std/utilities/tuple/tuple.tuple/tuple.assign/convert_copy.pass.cpp

8–9

Pre-existing: Please update the URL to llvm.org/PR45879

[ldionne]

Oh, thanks, I was literally going to land the same change! I assume you noticed the Bootstrapping build was failing?

@haowei Just for the sake of the process, in the future, please try to follow the libc++ review guidelines and wait for the libc++ review group to be green before committing. You can always ping us on Discord if you're missing some approvals on a simple (and time critical) change like this, and we'll be happy to take a look quickly. Thanks for the patch!

[tstellar]

Is this something I should be updating when I bump the version in trunk?

[haowei]

In D118838#3292088, @ldionne wrote:

Oh, thanks, I was literally going to land the same change! I assume you noticed the Bootstrapping build was failing?

@haowei Just for the sake of the process, in the future, please try to follow the libc++ review guidelines and wait for the libc++ review group to be green before committing. You can always ping us on Discord if you're missing some approvals on a simple (and time critical) change like this, and we'll be happy to take a look quickly. Thanks for the patch!

Oh, sorry I will follow the guideline next time. I thought the review process were the same.

I saw breakages on Fuchsia's clang linux builders due this test after the version was bumped to 15. And I found it out this test was broken since clang-13. So I just added clang-15 since the underlying bug was not yet fixed.

[haowei]

In D118838#3292100, @tstellar wrote:

Is this something I should be updating when I bump the version in trunk?

I think the problem is that this test was banned in clang-13 already but the underlying bug was never fixed. So whenever you bump the clang version number, if this file was not updated, it will fail again. We might want to fix the underlying bug so we don't have to add another exception here each time the version was bumped.

[ldionne]

In D118838#3292100, @tstellar wrote:

Is this something I should be updating when I bump the version in trunk?

No, I don't think so. As @haowei said, we want to fix the underlying bug instead. It's actually good that it breaks every time, because it reminds us that this hasn't been fixed yet (and in this case I pinged D103395).

In D118838#3292122, @haowei wrote:

Oh, sorry I will follow the guideline next time. I thought the review process were the same.

Yup, no worries at all!

libcxx/test/std/utilities/tuple/tuple.tuple/tuple.assign/convert_copy.pass.cpp

 //===----------------------------------------------------------------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
 
-// Triggers a Clang assertion: https://bugs.llvm.org/show_bug.cgi?id=45879
-// UNSUPPORTED: clang-13, clang-14
+// Triggers a Clang assertion: llvm.org/PR45879
+// UNSUPPORTED: clang-13, clang-14, clang-15
 
 // <tuple>
 
 // template <class... Types> class tuple;
 
 // template <class... UTypes>
 //   tuple& operator=(const tuple<UTypes...>& u);