From: Louis Dionne Date: Tue, 23 Apr 2019 14:05:04 +0000 (+0000) Subject: [libc++] Remove redundant conditionals for Apple platforms X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d524fe5b3a0975a1c037669ade35c182681c14f9;p=platform%2Fupstream%2Fllvm.git [libc++] Remove redundant conditionals for Apple platforms Summary: In a bunch of places, we used to check whether LIBCXX_BUILDING_LIBCXXABI is defined OR we're building for an Apple platform. This used to be necessary in a time when Apple's build script did NOT define LIBCXX_BUILDING_LIBCXXABI. However this is not relevant anymore since Apple's build does define LIBCXX_BUILDING_LIBCXXABI. Reviewers: EricWF Subscribers: christof, jkorous, dexonsmith, libcxx-commits Tags: #libc Differential Revision: https://reviews.llvm.org/D60842 llvm-svn: 358988 --- diff --git a/libcxx/src/exception.cpp b/libcxx/src/exception.cpp index ee9f384..fce6db7 100644 --- a/libcxx/src/exception.cpp +++ b/libcxx/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 using namespace __cxxabiv1; #define HAVE_DEPENDENT_EH_ABI 1 diff --git a/libcxx/src/support/runtime/stdexcept_default.ipp b/libcxx/src/support/runtime/stdexcept_default.ipp index 91dd54b..c827ca4 100644 --- a/libcxx/src/support/runtime/stdexcept_default.ipp +++ b/libcxx/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 #endif diff --git a/libcxx/src/typeinfo.cpp b/libcxx/src/typeinfo.cpp index 5b893b1..49a07ef 100644 --- a/libcxx/src/typeinfo.cpp +++ b/libcxx/src/typeinfo.cpp @@ -45,11 +45,11 @@ size_t std::type_info::hash_code() const _NOEXCEPT { } #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() {