Skip to content

Commit 53010fc

Browse files
ldionnetru
authored andcommitted
[libc++] Fix AppleClang version number when checking for __builtin_verbose_trap support (#110161)
We should have been checking against 1700, not 17000, which was a typo. (cherry picked from commit 1eba879)
1 parent 8679d1b commit 53010fc

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

libcxx/vendor/llvm/default_assertion_handler.in

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,8 @@
2626
# if __has_builtin(__builtin_verbose_trap)
2727
// AppleClang shipped a slightly different version of __builtin_verbose_trap from the upstream
2828
// version before upstream Clang actually got the builtin.
29-
# if defined(_LIBCPP_APPLE_CLANG_VER) && _LIBCPP_APPLE_CLANG_VER < 17000
29+
// TODO: Remove once AppleClang supports the two-arguments version of the builtin.
30+
# if defined(_LIBCPP_APPLE_CLANG_VER) && _LIBCPP_APPLE_CLANG_VER < 1700
3031
# define _LIBCPP_ASSERTION_HANDLER(message) __builtin_verbose_trap(message)
3132
# else
3233
# define _LIBCPP_ASSERTION_HANDLER(message) __builtin_verbose_trap("libc++", message)

0 commit comments

Comments
 (0)