[libcxx-commits] [libcxx] [libc++] Remove _LIBCPP_HAS_SIZED_DEALLOCATION (PR #172285)

[Nikolas Klauser via libcxx-commits]

================
@@ -13,7 +13,6 @@
 #include <__cstddef/max_align_t.h>
 #include <__cstddef/size_t.h>
 #include <__new/align_val_t.h>
-#include <__new/global_new_delete.h> // for _LIBCPP_HAS_SIZED_DEALLOCATION
----------------
philnik777 wrote:

I kinda expected that we wouldn't have to provide declarations for a builtin. A quick test on godbolt seems to support that assumption: https://godbolt.org/z/dxozozn6M. Do you maybe have a full reproducer?

FWIW it looks to me like it does find the overload (`note: candidate function not viable: no known conversion from 'align_val_t' to 'std::align_val_t' for 3rd argument`) but somehow thinks that there are two `align_val_t`s?

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