Stream: git-wasmtime

Topic: wasmtime / PR #9904 pulley: Implement 16x8 arithmetics

view this post on Zulip Wasmtime GitHub notifications bot (Dec 27 2024 at 04:31):

eagr opened PR #9904 from eagr:16x8-arth to bytecodealliance:main:

Helps #9783

view this post on Zulip Wasmtime GitHub notifications bot (Dec 27 2024 at 04:31):

eagr requested cfallin for a review on PR #9904.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 27 2024 at 04:31):

eagr requested wasmtime-compiler-reviewers for a review on PR #9904.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 27 2024 at 04:31):

eagr requested dicej for a review on PR #9904.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 27 2024 at 04:31):

eagr requested wasmtime-core-reviewers for a review on PR #9904.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 27 2024 at 04:31):

eagr requested wasmtime-default-reviewers for a review on PR #9904.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 27 2024 at 05:45):

github-actions[bot] commented on PR #9904:

Subscribe to Label Action

cc @fitzgen

<details>
This issue or pull request has been labeled: "cranelift", "pulley"

Thus the following users have been cc'd because of the following labels:

To subscribe or unsubscribe from this label, edit the <code>.github/subscribe-to-label.json</code> configuration file.

Learn more.
</details>

view this post on Zulip Wasmtime GitHub notifications bot (Dec 27 2024 at 05:48):

alexcrichton submitted PR review:

Thanks for this!

view this post on Zulip Wasmtime GitHub notifications bot (Dec 27 2024 at 05:48):

alexcrichton created PR review comment:

Could this perhaps be written as *a = (u32::from(a) + u32::from(b) + 1) / 2? Basically the more classical version of an average which explicitly takes advantage of wider-precision arithmetic to avoid overflow issues.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 27 2024 at 05:57):

eagr submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 27 2024 at 05:57):

eagr created PR review comment:

instantly convinced :))

view this post on Zulip Wasmtime GitHub notifications bot (Dec 27 2024 at 06:31):

eagr updated PR #9904.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 27 2024 at 06:50):

eagr updated PR #9904.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 27 2024 at 07:00):

eagr updated PR #9904.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 28 2024 at 00:35):

alexcrichton merged PR #9904.

Last updated: Jan 24 2025 at 00:11 UTC