On 11/29/21 3:27 PM, Frédéric Pétrot wrote:
On 29/11/2021 11:07, Richard Henderson wrote:
On 11/28/21 2:57 PM, Frédéric Pétrot wrote:
--- /dev/null
+++ b/util/int128.c
@@ -0,0 +1,145 @@
+#include "qemu/osdep.h"
+#include "qemu/host-utils.h"
+#include "qemu/int128.h"
Missing file header and copyright boilerplate.
+#ifdef CONFIG_INT128
+
+Int128 int128_divu(Int128 a, Int128 b)
+{
+ return (__uint128_t)a / (__uint128_t)b;
+}
+
+Int128 int128_remu(Int128 a, Int128 b)
+{
+ return (__uint128_t)a % (__uint128_t)b;
+}
+
+Int128 int128_divs(Int128 a, Int128 b)
+{
+ return a / b;
+}
+
+Int128 int128_rems(Int128 a, Int128 b)
+{
+ return a % b;
+}
I think we should simply expose these inline, and let the compiler call its runtime
function directly.
Thanks.
Ok, I'll drop that and handle the CONFIG_INT128 directly in the rv128
div/rem helpers then.
No, that's not what I meant. Copy these directly into include/qemu/int128.h and add
static inline, within the existing CONFIG_INT128 block.
r~
