Skip to content

Commit b2409ee

Browse files
committed
[libc++] std::cmp_less and other integer comparison functions could be improved with _LIBCPP_ASSUME
1 parent d98f191 commit b2409ee

File tree

1 file changed

+5
-4
lines changed
  • libcxx/include/__utility

1 file changed

+5
-4
lines changed

libcxx/include/__utility/cmp.h

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
#ifndef _LIBCPP___UTILITY_CMP_H
1010
#define _LIBCPP___UTILITY_CMP_H
1111

12+
#include "__assert"
1213
#include <__config>
1314
#include <__type_traits/integer_traits.h>
1415
#include <__type_traits/is_signed.h>
@@ -29,10 +30,10 @@ _LIBCPP_BEGIN_NAMESPACE_STD
2930
template <__signed_or_unsigned_integer _Tp, __signed_or_unsigned_integer _Up>
3031
_LIBCPP_HIDE_FROM_ABI constexpr bool cmp_equal(_Tp __t, _Up __u) noexcept {
3132
if constexpr (sizeof(_Tp) < sizeof(int) && sizeof(_Up) < sizeof(int)) {
32-
__builtin_assume(__t < numeric_limits<int>::max() && __u < numeric_limits<int>::max());
33+
_LIBCPP_ASSUME(__t < numeric_limits<int>::max() && __u < numeric_limits<int>::max());
3334
return static_cast<int>(__t) == static_cast<int>(__u);
3435
} else if constexpr (sizeof(_Tp) < sizeof(long long) && sizeof(_Up) < sizeof(long long)) {
35-
__builtin_assume(__t < numeric_limits<long long>::max() && __u < numeric_limits<long long>::max());
36+
_LIBCPP_ASSUME(__t < numeric_limits<long long>::max() && __u < numeric_limits<long long>::max());
3637
return static_cast<long long>(__t) == static_cast<long long>(__u);
3738
} else if constexpr (is_signed_v<_Tp> == is_signed_v<_Up>)
3839
return __t == __u;
@@ -50,10 +51,10 @@ _LIBCPP_HIDE_FROM_ABI constexpr bool cmp_not_equal(_Tp __t, _Up __u) noexcept {
5051
template <__signed_or_unsigned_integer _Tp, __signed_or_unsigned_integer _Up>
5152
_LIBCPP_HIDE_FROM_ABI constexpr bool cmp_less(_Tp __t, _Up __u) noexcept {
5253
if constexpr (sizeof(_Tp) < sizeof(int) && sizeof(_Up) < sizeof(int)) {
53-
__builtin_assume(__t < numeric_limits<int>::max() && __u < numeric_limits<int>::max());
54+
_LIBCPP_ASSUME(__t < numeric_limits<int>::max() && __u < numeric_limits<int>::max());
5455
return static_cast<int>(__t) < static_cast<int>(__u);
5556
} else if constexpr (sizeof(_Tp) < sizeof(long long) && sizeof(_Up) < sizeof(long long)) {
56-
__builtin_assume(__t < numeric_limits<long long>::max() && __u < numeric_limits<long long>::max());
57+
_LIBCPP_ASSUME(__t < numeric_limits<long long>::max() && __u < numeric_limits<long long>::max());
5758
return static_cast<long long>(__t) < static_cast<long long>(__u);
5859
} else if constexpr (is_signed_v<_Tp> == is_signed_v<_Up>)
5960
return __t < __u;

0 commit comments

Comments
 (0)