Skip to content

wasm-smith: generate shared memory when --threads is present#624

Merged
alexcrichton merged 5 commits intobytecodealliance:mainfrom
abrown:shared-memory
Jun 8, 2022
Merged

wasm-smith: generate shared memory when --threads is present#624
alexcrichton merged 5 commits intobytecodealliance:mainfrom
abrown:shared-memory

Conversation

@abrown
Copy link
Member

@abrown abrown commented Jun 7, 2022

One of the parts to the threads proposal is the addition of shared
memory. This change allows wasm-smith to generate Wasm code using
memory that is shared; it does not include the other parts of the
threads proposal (atomics, wait/notify).

abrown added 2 commits June 7, 2022 16:15
One of the parts to the threads proposal is the addition of shared
memory. This change allows `wasm-smith` to generate Wasm code using
memory that is shared; it does not include the other parts of the
threads proposal (atomics, `wait/notify`).
Copy link
Member

@alexcrichton alexcrichton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems reasonable to me!

Copy link
Member

@fitzgen fitzgen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Looks good, modulo what Alex already pointed out, and another little thing below.

@abrown abrown marked this pull request as ready for review June 8, 2022 18:09
@alexcrichton alexcrichton merged commit f9cd94e into bytecodealliance:main Jun 8, 2022
@abrown abrown deleted the shared-memory branch June 8, 2022 21:08
code-terror pushed a commit to code-terror/wasm-tools that referenced this pull request Aug 24, 2022
…odealliance#624)

* wasm-smith: generate shared memory when `--threads` is present

One of the parts to the threads proposal is the addition of shared
memory. This change allows `wasm-smith` to generate Wasm code using
memory that is shared; it does not include the other parts of the
threads proposal (atomics, `wait/notify`).

* fix: doc tests

* review: use 'Unstructured::ratio'

* review: add 'threads_enabled' to 'SwarmConfig'

* review: disable threads in 'SwarmConfig'
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants