From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 2181) id 818773888019; Thu, 24 Jun 2021 13:34:28 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 818773888019 MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="utf-8" From: Jonathan Wakely To: gcc-cvs@gcc.gnu.org, libstdc++-cvs@gcc.gnu.org Subject: [gcc r12-1783] libstdc++: Remove incorrect fabs(simd) overload X-Act-Checkin: gcc X-Git-Author: Matthias Kretz X-Git-Refname: refs/heads/master X-Git-Oldrev: 343f01f4cde17003b6721722361e190bd6cffd15 X-Git-Newrev: dd1c7792d6ff70e830e0ff8028bbac8379f5a79c Message-Id: <20210624133428.818773888019@sourceware.org> Date: Thu, 24 Jun 2021 13:34:28 +0000 (GMT) X-BeenThere: libstdc++-cvs@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Libstdc++-cvs mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 24 Jun 2021 13:34:28 -0000 https://gcc.gnu.org/g:dd1c7792d6ff70e830e0ff8028bbac8379f5a79c commit r12-1783-gdd1c7792d6ff70e830e0ff8028bbac8379f5a79c Author: Matthias Kretz Date: Thu Jun 24 14:20:14 2021 +0100 libstdc++: Remove incorrect fabs(simd) overload fabs(int) returns double, this one didn't. This overload is not specified in the Parallelism TS 2. Also remove the comment about labs and llabs: it doesn't belong here. Signed-off-by: Matthias Kretz libstdc++-v3/ChangeLog: * include/experimental/bits/simd_math.h (fabs): Remove fabs(simd) overload. Diff: --- libstdc++-v3/include/experimental/bits/simd_math.h | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/libstdc++-v3/include/experimental/bits/simd_math.h b/libstdc++-v3/include/experimental/bits/simd_math.h index d954e761eee..fe670e60961 100644 --- a/libstdc++-v3/include/experimental/bits/simd_math.h +++ b/libstdc++-v3/include/experimental/bits/simd_math.h @@ -865,22 +865,6 @@ template abs(const simd<_Tp, _Abi>& __x) { return {__private_init, _Abi::_SimdImpl::_S_abs(__data(__x))}; } -template - enable_if_t && is_signed_v<_Tp>, simd<_Tp, _Abi>> - fabs(const simd<_Tp, _Abi>& __x) - { return {__private_init, _Abi::_SimdImpl::_S_abs(__data(__x))}; } - -// the following are overloads for functions in and not covered by -// [parallel.simd.math]. I don't see much value in making them work, though -/* -template simd labs(const simd &__x) -{ return {__private_init, _Abi::_SimdImpl::abs(__data(__x))}; } - -template simd llabs(const simd -&__x) -{ return {__private_init, _Abi::_SimdImpl::abs(__data(__x))}; } -*/ - #define _GLIBCXX_SIMD_CVTING2(_NAME) \ template \ _GLIBCXX_SIMD_INTRINSIC simd<_Tp, _Abi> _NAME( \