Stream: git-wasmtime

Topic: wasmtime / issue #4771 cranelift: Disable i128 divs on fu...

view this post on Zulip Wasmtime GitHub notifications bot (Aug 24 2022 at 16:55):

bjorn3 commented on issue #4771:

For reference cg_clif uses compiler-builtins intrinsics for 128bit multiplication and division.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 24 2022 at 16:59):

afonso360 commented on issue #4771:

I think we already support i128 multiplications (at least the test case is enabled), but I'd still like to implement i128 div/rem and cleanup cg_clif's codegen!

view this post on Zulip Wasmtime GitHub notifications bot (Aug 24 2022 at 17:08):

bjorn3 commented on issue #4771:

Won't 128bit div and rem be rather large? Using compiler-builtins would avoid code bloat in those cases.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 24 2022 at 17:08):

bjorn3 edited a comment on issue #4771:

Won't 128bit div and rem be rather large? Using compiler-builtins avoids code bloat in those cases.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 24 2022 at 17:10):

afonso360 commented on issue #4771:

Well the plan was to do exactly that, lower them as libcalls, or are they not available in all platforms?

view this post on Zulip Wasmtime GitHub notifications bot (Aug 24 2022 at 17:13):

bjorn3 commented on issue #4771:

I see. I believe it is available on all platforms. However LLVM has changed the abi of 128bit compiler builtins on windows before and this may happen again in the future.

Last updated: Dec 23 2024 at 13:07 UTC