From: Jonathan Wakely Date: Mon, 14 May 2018 15:35:12 +0000 (+0100) Subject: Add __attribute__((malloc) to allocator and remove unused code X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=cd2f262b2fa4292a39b97ab590b46e79d349817c;p=gcc.git Add __attribute__((malloc) to allocator and remove unused code * include/bits/valarray_array.h (__valarray_get_memory): Remove. (__valarray_get_storage): Call operator new directly. Remove ignored top-level restrict qualifier and add malloc attribute instead. (_Array<_Tp>::_Array(size_t)): Remove unused constructor. From-SVN: r260230 --- diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog index 9d47d008147..e8322137756 100644 --- a/libstdc++-v3/ChangeLog +++ b/libstdc++-v3/ChangeLog @@ -1,5 +1,10 @@ 2018-05-14 Jonathan Wakely + * include/bits/valarray_array.h (__valarray_get_memory): Remove. + (__valarray_get_storage): Call operator new directly. Remove ignored + top-level restrict qualifier and add malloc attribute instead. + (_Array<_Tp>::_Array(size_t)): Remove unused constructor. + PR libstdc++/67554 * include/bits/valarray_array.h (_Array_copy_ctor<_Tp, true>) (_Array_copier<_Tp, true>): Do not pass null pointers to memcpy. diff --git a/libstdc++-v3/include/bits/valarray_array.h b/libstdc++-v3/include/bits/valarray_array.h index 07f38ed03ed..6759d6003e9 100644 --- a/libstdc++-v3/include/bits/valarray_array.h +++ b/libstdc++-v3/include/bits/valarray_array.h @@ -47,18 +47,15 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION // Helper functions on raw pointers // - // We get memory by the old fashion way - inline void* - __valarray_get_memory(size_t __n) - { return operator new(__n); } + // We get memory the old fashioned way + template + _Tp* + __valarray_get_storage(size_t) __attribute__((__malloc__)); template - inline _Tp*__restrict__ + inline _Tp* __valarray_get_storage(size_t __n) - { - return static_cast<_Tp*__restrict__> - (std::__valarray_get_memory(__n * sizeof(_Tp))); - } + { return static_cast<_Tp*>(operator new(__n * sizeof(_Tp))); } // Return memory to the system inline void @@ -410,7 +407,6 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION template struct _Array { - explicit _Array(size_t); explicit _Array(_Tp* const __restrict__); explicit _Array(const valarray<_Tp>&); _Array(const _Tp* __restrict__, size_t); @@ -503,12 +499,6 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION __dst._M_data, __j._M_data); } - template - inline - _Array<_Tp>::_Array(size_t __n) - : _M_data(__valarray_get_storage<_Tp>(__n)) - { std::__valarray_default_construct(_M_data, _M_data + __n); } - template inline _Array<_Tp>::_Array(_Tp* const __restrict__ __p)