[libcxx-commits] [libcxx] [libc++] Fix missing 'get_new_handler()' for Windows builds (PR #150182)

[via libcxx-commits]

================
@@ -11,9 +11,7 @@
 #include "include/atomic_support.h"
 
 #if defined(_LIBCPP_ABI_MICROSOFT)
-#  if !defined(_LIBCPP_ABI_VCRUNTIME)
-#    define _LIBPCPP_DEFINE_NEW_HANDLER
-#  endif
+#  define _LIBPCPP_DEFINE_NEW_HANDLER
----------------
siradam7th wrote:

> This doesn't seem right. The definitions provided `__new_handler` are dummy and non-conformant. Perhaps we should rely on VCRuntime's functionalities.

Care to elaborate? from what I've seen in MS-STL they both have similar implementations.

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