abrown opened PR #5858 from fix-threads-test
to main
:
As @yamt points out [here], the
wait
/notify
pairing used in this manual WAT test was not effective. Thewait
always immediately returned, meaning that the main thread essentially spins until a counter is atomically incremented. This is fine for test correctness, but was not the original intent, which was lost in a refactoring. This change uses the$i
local to keep track of the counter value we expect to see for thewait
, so that thewait
/notify
pair actually waits as expected.[here]: https://github.com/bytecodealliance/wasmtime/pull/5484#discussion_r1101200012
<!--
Please ensure that the following steps are all taken care of before submitting
the PR.
[ ] This has been discussed in issue #..., or if not, please tell us why
here.[ ] A short description of what this does, why it is needed; if the
description becomes long, the matter should probably be discussed in an issue
first.[ ] This PR contains test cases, if meaningful.
- [ ] A reviewer from the core maintainer team has been assigned for this PR.
If you don't know who could review this, please indicate so. The list of
suggested reviewers on the right can help you.Please ensure all communication adheres to the code of conduct.
-->
alexcrichton submitted PR review.
alexcrichton merged PR #5858.
Last updated: Dec 23 2024 at 13:07 UTC