[libcxx-commits] [libcxx] [libcxxabi] [libc++][libc++abi] Build with -fsized-deallocation (PR #96217)

[via libcxx-commits]

================
@@ -247,6 +247,10 @@ endif()
 add_compile_flags("${LIBCXXABI_ADDITIONAL_COMPILE_FLAGS}")
 add_library_flags("${LIBCXXABI_ADDITIONAL_LIBRARIES}")
 
+# Build with -fsized-deallocation, which is default in recent versions of Clang.
+# TODO(LLVM 21): This can be dropped once we only support Clang >= 19.
+add_compile_flags(-fsized-deallocation)
----------------
EricWF wrote:

I don't think this change should affect anything. Can we drop it and see what happens?

The size-deallocation functions should always be present in the library, and we shouldn't be calling new/delete from inside the library (I think?)



https://github.com/llvm/llvm-project/pull/96217