[libcxx-commits] [libcxx] r358518 - [libc++] Remove old workaround for buildit

Louis Dionne via libcxx-commits libcxx-commits at lists.llvm.org
Tue Apr 16 12:26:56 PDT 2019


Author: ldionne
Date: Tue Apr 16 12:26:56 2019
New Revision: 358518

URL: http://llvm.org/viewvc/llvm-project?rev=358518&view=rev
Log:
[libc++] Remove old workaround for buildit

Summary:
I'm not sure what the problem was at the time, however I don't think
this is necessary since buildit doesn't exist anymore.

Instead of the workaround, the correct thing to do is to leave out
the get_new_handler/set_new_handler definitions from libc++ when
we're getting them from libc++abi.

Reviewers: EricWF

Subscribers: christof, jkorous, dexonsmith, libcxx-commits

Tags: #libc

Differential Revision: https://reviews.llvm.org/D60717

Modified:
    libcxx/trunk/src/new.cpp

Modified: libcxx/trunk/src/new.cpp
URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/src/new.cpp?rev=358518&r1=358517&r2=358518&view=diff
==============================================================================
--- libcxx/trunk/src/new.cpp (original)
+++ libcxx/trunk/src/new.cpp Tue Apr 16 12:26:56 2019
@@ -22,12 +22,8 @@
 #   include "support/runtime/new_handler_fallback.ipp"
 #elif defined(__GLIBCXX__)
     // nothing to do
-#else
-#   if defined(__APPLE__) && !defined(_LIBCPP_BUILDING_HAS_NO_ABI_LIBRARY)
-#       include <cxxabi.h> // FIXME: remove this once buildit is gone.
-#   else
-#       include "support/runtime/new_handler_fallback.ipp"
-#   endif
+#elif !defined(_LIBCPP_DISABLE_NEW_DELETE_DEFINITIONS)
+#   include "support/runtime/new_handler_fallback.ipp"
 #endif
 
 namespace std




More information about the libcxx-commits mailing list