pcwang-thead added inline comments.
================
Comment at:
libcxx/test/std/language.support/support.dynamic/new.delete/new.delete.array/sized_delete_array14.pass.cpp:17
+// XFAIL: use_system_cxx_lib && target={{.+}}-apple-macosx10.{{9|10|11}}
#include <new>
----------------
Mordante wrote:
> This `// XFAIL: clang-12, clang-13` is still needed but should be `// XFAIL:
> clang-13, clang-14`. (Since the LLVM 14 release we only support these two
> versions.)
>
> For testing it's the easiest to remove line 147 of
> `libcxx/utils/ci/buildkite-pipeline.yml`
> https://github.com/llvm/llvm-project/blob/afcc6baac52fcc91d1636f6803f5c230e7018016/libcxx/utils/ci/buildkite-pipeline.yml#L147
>
> That way all builds run. The `Bootstrapping build` builds clang and uses that
> clang to test libc++. That way we can validate which builds fail and which
> succeed. Maybe some more builds need to be (temporary) disabled.
>
> Once we verify that works we need to:
> - undo the buildkite changes
> - disable this test temporary (`UNSUPPORTED: clang-15`)
> - create a followup patch to reenable the test
> After the change has landed it will take some time before the change is
> propagated to the CI. Once it's propagated the followup patch can be landed.
> I'm willing to create the followup patch and land it at the proper time.
>
Thanks! I will have a try later. :)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D112921/new/
https://reviews.llvm.org/D112921
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
