Commit c76e9f22912e3b987a4e4590eb386477ae114f85
2020-12-18T17:57:46
Merge pull request #5742 from lhchavez/fix-clang-32-bit-build Avoid using `__builtin_mul_overflow` with the clang+32-bit combo
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38
diff --git a/src/integer.h b/src/integer.h
index 026a1fa..28122a9 100644
--- a/src/integer.h
+++ b/src/integer.h
@@ -79,8 +79,12 @@ GIT_INLINE(int) git__is_int(long long p)
# define git__add_int64_overflow(out, one, two) \
__builtin_add_overflow(one, two, out)
-# define git__multiply_int64_overflow(out, one, two) \
- __builtin_mul_overflow(one, two, out)
+
+/* clang on 32-bit systems produces an undefined reference to `__mulodi4`. */
+# if !defined(__clang__) || !defined(GIT_ARCH_32)
+# define git__multiply_int64_overflow(out, one, two) \
+ __builtin_mul_overflow(one, two, out)
+# endif
/* Use Microsoft's safe integer handling functions where available */
#elif defined(_MSC_VER)
@@ -156,6 +160,10 @@ GIT_INLINE(bool) git__add_int64_overflow(int64_t *out, int64_t one, int64_t two)
return false;
}
+#endif
+
+/* If we could not provide an intrinsic implementation for this, provide a (slow) fallback. */
+#if !defined(git__multiply_int64_overflow)
GIT_INLINE(bool) git__multiply_int64_overflow(int64_t *out, int64_t one, int64_t two)
{
if ((one == -1 && two == INT_MIN) ||
@@ -166,7 +174,6 @@ GIT_INLINE(bool) git__multiply_int64_overflow(int64_t *out, int64_t one, int64_t
*out = one * two;
return false;
}
-
#endif
#endif