fix bug of a failed test on macOS

guuzaa · Mar 23, 2024 · 815d91c · 815d91c
1 parent 6f68832
commit 815d91c
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/src/include/int128_have_intrinstic.inc b/src/include/int128_have_intrinstic.inc
@@ -120,9 +120,15 @@ constexpr int128 operator-(int128 lhs, int128 rhs) { return static_cast<__int128
 
 inline int128 operator*(int128 lhs, int128 rhs) { return static_cast<__int128>(lhs) * static_cast<__int128>(rhs); }
 
-inline int128 operator/(int128 lhs, int128 rhs) { return static_cast<__int128>(lhs) / static_cast<__int128>(rhs); }
+inline int128 operator/(int128 lhs, int128 rhs) {
+  assert(rhs != 0);
+  return static_cast<__int128>(lhs) / static_cast<__int128>(rhs);
+}
 
-inline int128 operator%(int128 lhs, int128 rhs) { return static_cast<__int128>(lhs) % static_cast<__int128>(rhs); }
+inline int128 operator%(int128 lhs, int128 rhs) {
+  assert(rhs != 0);
+  return static_cast<__int128>(lhs) % static_cast<__int128>(rhs);
+}
 
 inline int128 int128::operator++(int) {
   int128 tmp(*this);

diff --git a/src/numbers/int128.cc b/src/numbers/int128.cc
@@ -191,7 +191,7 @@ int128 operator%(int128 lhs, int128 rhs) {
   }
   return make_int128(int128_internal::BitCastToSigned(uint128_high64(remainder)), uint128_low64(remainder));
 }
-#endif  // NUMBERS_HAVE_INTRINSTIC_INT128
+#endif  // ! NUMBERS_HAVE_INTRINSTIC_INT128
 
 std::ostream &operator<<(std::ostream &os, int128 v) {
   std::ios_base::fmtflags flags = os.flags();