[libcxx-commits] [PATCH] D112921: [clang] Enable sized deallocation by default in C++14 onwards

[Wang Pengcheng via Phabricator via libcxx-commits]

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