[libcxx-commits] [PATCH] D60842: [libc++] Remove redundant conditionals for Apple platforms

[Louis Dionne via Phabricator via libcxx-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL358988: [libc++] Remove redundant conditionals for Apple platforms (authored by ldionne, committed by ).
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

Changed prior to commit:
  https://reviews.llvm.org/D60842?vs=195642&id=196243#toc

Repository:
  rL LLVM

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

https://reviews.llvm.org/D60842

Files:
  libcxx/trunk/src/exception.cpp
  libcxx/trunk/src/support/runtime/stdexcept_default.ipp
  libcxx/trunk/src/typeinfo.cpp


Index: libcxx/trunk/src/exception.cpp
===================================================================
--- libcxx/trunk/src/exception.cpp
+++ libcxx/trunk/src/exception.cpp
@@ -10,8 +10,7 @@
 #include "new"
 #include "typeinfo"
 
-#if defined(LIBCXXRT) || defined(LIBCXX_BUILDING_LIBCXXABI) || \
-  (defined(__APPLE__) && !defined(_LIBCPP_BUILDING_HAS_NO_ABI_LIBRARY))
+#if defined(LIBCXXRT) || defined(LIBCXX_BUILDING_LIBCXXABI)
   #include <cxxabi.h>
   using namespace __cxxabiv1;
   #define HAVE_DEPENDENT_EH_ABI 1
Index: libcxx/trunk/src/typeinfo.cpp
===================================================================
--- libcxx/trunk/src/typeinfo.cpp
+++ libcxx/trunk/src/typeinfo.cpp
@@ -45,11 +45,11 @@
 }
 #endif // _LIBCPP_ABI_MICROSOFT
 
-// FIXME: Remove __APPLE__ default here once buildit is gone.
 // FIXME: Remove the _LIBCPP_BUILDING_HAS_NO_ABI_LIBRARY configuration.
-#if (!defined(LIBCXX_BUILDING_LIBCXXABI) && !defined(LIBCXXRT) &&              \
-     !defined(__GLIBCXX__) && !defined(__APPLE__) &&                           \
-     !defined(_LIBCPP_ABI_VCRUNTIME)) ||   \
+#if (!defined(LIBCXX_BUILDING_LIBCXXABI) &&               \
+     !defined(LIBCXXRT) &&                                \
+     !defined(__GLIBCXX__) &&                             \
+     !defined(_LIBCPP_ABI_VCRUNTIME)) ||                  \
     defined(_LIBCPP_BUILDING_HAS_NO_ABI_LIBRARY)
 std::type_info::~type_info()
 {
Index: libcxx/trunk/src/support/runtime/stdexcept_default.ipp
===================================================================
--- libcxx/trunk/src/support/runtime/stdexcept_default.ipp
+++ libcxx/trunk/src/support/runtime/stdexcept_default.ipp
@@ -10,8 +10,7 @@
 
 /* For _LIBCPPABI_VERSION */
 #if !defined(_LIBCPP_BUILDING_HAS_NO_ABI_LIBRARY) &&                           \
-    (defined(LIBCXX_BUILDING_LIBCXXABI) || defined(__APPLE__) ||               \
-     defined(LIBCXXRT))
+    (defined(LIBCXX_BUILDING_LIBCXXABI) || defined(LIBCXXRT))
 #include <cxxabi.h>
 #endif
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D60842.196243.patch
Type: text/x-patch
Size: 2007 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20190423/6f3d3456/attachment.bin>