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

Mark de Wever via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 14 09:15:10 PDT 2023


Mordante added inline comments.


================
Comment at: libcxx/test/std/language.support/support.dynamic/new.delete/new.delete.array/sized_delete_array14.pass.cpp:16
+// Sized deallocation was added in macOS 10.12 and aligned OSes.
+// XFAIL: use_system_cxx_lib && target={{.+}}-apple-macosx10.{{9|10|11}}
 
----------------
Maybe as a pragmatic solution
```
// TODO(mordante) fix this test after updating clang in Docker
// UNSUPPORTED: clang-17
```
The same for the other failing test. Then I'll fix this once the libc++ CI can be updated with this fix included.



================
Comment at: libcxx/utils/ci/buildkite-pipeline.yml:148
   #
-  - wait
+  # - wait
 
----------------
Please undo the changes to this file.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112921/new/

https://reviews.llvm.org/D112921



More information about the cfe-commits mailing list