[libcxx-commits] [PATCH] D57355: [libc++] Fix Windows build error in <functional>
Tom Anderson via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Tue Jan 29 15:19:51 PST 2019
This revision was automatically updated to reflect the committed changes.
Closed by commit rCXX352561: [libc++] Fix Windows build error in <functional> (authored by thomasanderson, committed by ).
Herald added subscribers: libcxx-commits, ldionne.
Repository:
rCXX libc++
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57355/new/
https://reviews.llvm.org/D57355
Files:
include/functional
test/support/nasty_macros.hpp
Index: include/functional
===================================================================
--- include/functional
+++ include/functional
@@ -1488,8 +1488,9 @@
_LIBCPP_INLINE_VISIBILITY
const _Target& __target() const { return __f_.first(); }
+ // WIN32 APIs may define __allocator, so use __get_allocator instead.
_LIBCPP_INLINE_VISIBILITY
- const _Alloc& __allocator() const { return __f_.second(); }
+ const _Alloc& __get_allocator() const { return __f_.second(); }
_LIBCPP_INLINE_VISIBILITY
explicit __alloc_func(_Target&& __f)
@@ -1611,7 +1612,7 @@
{
typedef allocator_traits<_Alloc> __alloc_traits;
typedef typename __rebind_alloc_helper<__alloc_traits, __func>::type _Ap;
- _Ap __a(__f_.__allocator());
+ _Ap __a(__f_.__get_allocator());
typedef __allocator_destructor<_Ap> _Dp;
unique_ptr<__func, _Dp> __hold(__a.allocate(1), _Dp(__a, 1));
::new ((void*)__hold.get()) __func(__f_.__target(), _Alloc(__a));
@@ -1622,7 +1623,7 @@
void
__func<_Fp, _Alloc, _Rp(_ArgTypes...)>::__clone(__base<_Rp(_ArgTypes...)>* __p) const
{
- ::new (__p) __func(__f_.__target(), __f_.__allocator());
+ ::new (__p) __func(__f_.__target(), __f_.__get_allocator());
}
template<class _Fp, class _Alloc, class _Rp, class ..._ArgTypes>
@@ -1638,7 +1639,7 @@
{
typedef allocator_traits<_Alloc> __alloc_traits;
typedef typename __rebind_alloc_helper<__alloc_traits, __func>::type _Ap;
- _Ap __a(__f_.__allocator());
+ _Ap __a(__f_.__get_allocator());
__f_.destroy();
__a.deallocate(this, 1);
}
@@ -1924,7 +1925,7 @@
typedef typename __rebind_alloc_helper<__alloc_traits, _Fun>::type
_FunAlloc;
_Fun* __f = static_cast<_Fun*>(__s);
- _FunAlloc __a(__f->__allocator());
+ _FunAlloc __a(__f->__get_allocator());
__f->destroy();
__a.deallocate(__f, 1);
}
Index: test/support/nasty_macros.hpp
===================================================================
--- test/support/nasty_macros.hpp
+++ test/support/nasty_macros.hpp
@@ -54,6 +54,7 @@
// Test that libc++ doesn't use names reserved by WIN32 API Macros.
// NOTE: Obviously we can only define these on non-windows platforms.
#ifndef _WIN32
+#define __allocator NASTY_MACRO
#define __deallocate NASTY_MACRO
#define __out NASTY_MACRO
#endif
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D57355.184192.patch
Type: text/x-patch
Size: 2361 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20190129/40708866/attachment.bin>
More information about the libcxx-commits
mailing list