<limits>: fix traps for integers

stl/inc/limits

@@ -120,7 +120,10 @@ struct _Num_int_base : _Num_base { // base for integer types
     static constexpr bool is_exact       = true;
     static constexpr bool is_integer     = true;
     static constexpr bool is_specialized = true;
-    static constexpr int radix           = 2;
+#if !defined(__clang__) || (defined(_M_X64) && !defined(_M_ARM64EC)) || defined(_M_IX86)
+    static constexpr bool traps = true;
+#endif
+    static constexpr int radix = 2;
 };
 
 struct _Num_float_base : _Num_base { // base for floating-point types
@@ -178,6 +181,7 @@ public:
         return 0;
     }
 
+    static constexpr bool traps = false;
     static constexpr int digits = 1;
 };
 
@@ -222,6 +226,7 @@ public:
 
     static constexpr bool is_signed = CHAR_MIN != 0;
     static constexpr bool is_modulo = CHAR_MIN == 0;
+    static constexpr bool traps     = false;
     static constexpr int digits     = 8 - (CHAR_MIN != 0);
     static constexpr int digits10   = 2;
 };
@@ -266,6 +271,7 @@ public:
     }
 
     static constexpr bool is_signed = true;
+    static constexpr bool traps     = false;
     static constexpr int digits     = 7;
     static constexpr int digits10   = 2;
 };
@@ -310,6 +316,7 @@ public:
     }
 
     static constexpr bool is_modulo = true;
+    static constexpr bool traps     = false;
     static constexpr int digits     = 8;
     static constexpr int digits10   = 2;
 };
@@ -355,6 +362,7 @@ public:
     }
 
     static constexpr bool is_modulo = true;
+    static constexpr bool traps     = false;
     static constexpr int digits     = 8;
     static constexpr int digits10   = 2;
 };
@@ -400,6 +408,7 @@ public:
     }
 
     static constexpr bool is_modulo = true;
+    static constexpr bool traps     = false;
     static constexpr int digits     = 16;
     static constexpr int digits10   = 4;
 };
@@ -444,6 +453,7 @@ public:
     }
 
     static constexpr bool is_modulo = true;
+    static constexpr bool traps     = false;
     static constexpr int digits     = 32;
     static constexpr int digits10   = 9;
 };
@@ -488,6 +498,7 @@ public:
     }
 
     static constexpr bool is_modulo = true;
+    static constexpr bool traps     = false;
     static constexpr int digits     = 16;
     static constexpr int digits10   = 4;
 };
@@ -532,6 +543,7 @@ public:
     }
 
     static constexpr bool is_signed = true;
+    static constexpr bool traps     = false;
     static constexpr int digits     = 15;
     static constexpr int digits10   = 4;
 };
@@ -710,6 +722,7 @@ public:
     }
 
     static constexpr bool is_modulo = true;
+    static constexpr bool traps     = false;
     static constexpr int digits     = 16;
     static constexpr int digits10   = 4;
 };

tests/libcxx/expected_results.txt

@@ -449,11 +449,6 @@ std/depr/depr.c.headers/tgmath_h.pass.cpp:2 FAIL
 
 
 # *** CLANG ISSUES, NOT YET ANALYZED ***
-# Not analyzed. Clang apparently defines platform macros differently from C1XX.
-# The test inspects `__x86_64__` and `__i386__`, which MSVC doesn't define.
-# SKIPPED because this fails for x64/x86 and passes for ARM64.
-std/language.support/support.limits/limits/numeric.limits.members/traps.pass.cpp:2 SKIPPED
-
 # Not analyzed. Possibly C++20 equality operator rewrite issues.
 std/utilities/expected/expected.expected/equality/equality.other_expected.pass.cpp:2 FAIL
 std/utilities/expected/expected.void/equality/equality.other_expected.pass.cpp:2 FAIL
@@ -714,6 +709,15 @@ std/iterators/iterator.requirements/iterator.assoc.types/readable.traits/indirec
 # Not analyzed, likely bogus test. constexpr fails with "vector iterators incompatible".
 std/ranges/range.adaptors/range.join.with/range.join.with.iterator/ctor.default.pass.cpp FAIL
 
+# Problems in this test:
+# - Clang does not trap on ARM64, but MSVC does.
+# - The test inspects `__x86_64__` and `__i386__`, which MSVC doesn't define.
+# - The __x86_64__ is defined on ARM64EC, but it is expected to behave like ARM64
+# :2 SKIPPED because this passes for x64/x86/ARM64 and fails for ARM64EC.
+std/language.support/support.limits/limits/numeric.limits.members/traps.pass.cpp:0 FAIL
+std/language.support/support.limits/limits/numeric.limits.members/traps.pass.cpp:1 FAIL
+std/language.support/support.limits/limits/numeric.limits.members/traps.pass.cpp:2 SKIPPED
+
 
 # *** LIKELY STL BUGS ***
 # Not analyzed, likely STL bugs. Various assertions.

tests/std/test.lst

@@ -279,6 +279,7 @@ tests\GH_005553_regex_character_translation
 tests\GH_005768_pow_accuracy
 tests\GH_005780_non_ascii_locales
 tests\GH_005800_stable_sort_large_alignment
+tests\GH_005816_numeric_limits_traps
 tests\GH_005968_headers_provide_begin_end
 tests\LWG2381_num_get_floating_point
 tests\LWG2510_tag_classes

tests/std/tests/GH_005816_numeric_limits_traps/env.lst

@@ -0,0 +1,4 @@
+# Copyright (c) Microsoft Corporation.
+# SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+
+RUNALL_INCLUDE ..\usual_matrix.lst

tests/std/tests/GH_005816_numeric_limits_traps/test.compile.pass.cpp

@@ -0,0 +1,56 @@
+// Copyright (c) Microsoft Corporation.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+
+#include <limits>
+#include <type_traits>
+
+template <class T>
+constexpr bool traps_ = std::numeric_limits<T>::traps;
+
+#if defined(_M_IX86) || (defined(_M_X64) && !defined(_M_ARM64EC))
+static_assert(traps_<int>,
+    "The #ED hardware exception always happens for zero division and for division overflow INT_MIN/-1. "
+    "These are translated to STATUS_INTEGER_DIVIDE_BY_ZERO and STATUS_INTEGER_OVERFLOW SEH exceptions");
+#elif defined(_M_ARM64) || defined(_M_ARM64EC) || defined(_M_HYBRID_X86_ARM64)
+#ifdef __clang__
+static_assert(!traps_<int>, "The hardware does not trap. Clang compiles code as is, so there's no trap");
+#else // ^^^ defined(__clang__) / !defined(__clang__) vvv
+static_assert(traps_<int>, "The hardware does not trap. MSVC inserts a check for zero to trap zero division. "
+                           "It does not insert checks for INT_MIN/-1 division overflow though");
+#endif // ^^^ !defined(__clang__) ^^^
+#else // ^^^ defined(_M_ARM64) || defined(_M_ARM64EC) || defined(_M_HYBRID_X86_ARM64) ^^^
+#error Unsupported hardware
+#endif
+
+template <class T>
+constexpr bool non_promoted_and_traps_if_int_does = std::is_same_v<decltype(~T{}), T> && traps_<T> == traps_<int>;
+
+static_assert(non_promoted_and_traps_if_int_does<unsigned int> //
+                  && non_promoted_and_traps_if_int_does<long> //
+                  && non_promoted_and_traps_if_int_does<unsigned long> //
+                  && non_promoted_and_traps_if_int_does<long long> //
+                  && non_promoted_and_traps_if_int_does<unsigned long long>,
+    "all non-promoted integers should trap or not trap equally");
+
+template <class T>
+constexpr bool promoted_and_does_not_trap = !std::is_same_v<decltype(~T{}), T> && !traps_<T>;
+
+static_assert(!traps_<bool>, "bool does not trap for a moot reason; see LWG-554 resolution");
+
+static_assert(promoted_and_does_not_trap<char> //
+                  && promoted_and_does_not_trap<signed char> //
+                  && promoted_and_does_not_trap<unsigned char> //
+                  && promoted_and_does_not_trap<short> //
+                  && promoted_and_does_not_trap<unsigned short> //
+                  && promoted_and_does_not_trap<wchar_t> //
+                  && promoted_and_does_not_trap<char16_t> //
+                  && promoted_and_does_not_trap<char32_t>,
+    "promoted integers do not trap for a moot reason; see LWG-554 resolution");
+
+#ifdef __cpp_char8_t
+static_assert(
+    promoted_and_does_not_trap<char8_t>, "promoted integers do not trap for a moot reason; see LWG-554 resolution");
+#endif
+
+static_assert(!traps_<float> && !traps_<double> && !traps_<long double>,
+    "floats don't trap because even if '/fp:except' is passed, it should be enabled at runtime");