Skip to content
This repository was archived by the owner on Apr 25, 2025. It is now read-only.

Revert "Merging the entire reference-types repo (#96)" #115

Merged
merged 1 commit into from
Jun 15, 2020

Conversation

aheejin
Copy link
Member

@aheejin aheejin commented Jun 12, 2020

This reverts commit 10d6c6c.

This reverts #96, which squashed all upstream commits into one, which
was not the recommended way in
https://github.com/WebAssembly/proposals/blob/master/howto.md#syncing-with-upstream-changes.

This leaves README.md untouched because #99 made significant changes on
top of the merged version already.

cc @ioannad

This reverts commit 10d6c6c.

This reverts WebAssembly#96, which squashed all upstream commits into one, which
was not the recommended way in
https://github.com/WebAssembly/proposals/blob/master/howto.md#syncing-with-upstream-changes.

This leaves README.md untouched because WebAssembly#99 made significant changes on
top of the merged version already.
@aheejin aheejin requested review from Ms2ger and rossberg June 12, 2020 23:12
Copy link
Member

@rossberg rossberg left a comment

Choose a reason for hiding this comment

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

Rubberstamped

@aheejin aheejin merged commit 3c28153 into WebAssembly:master Jun 15, 2020
@aheejin aheejin deleted the revert_96 branch June 15, 2020 17:27
aheejin pushed a commit to aheejin/exception-handling that referenced this pull request Jun 15, 2020
Implement bulk memory operations in small-step semantics, as in spec.

Minor fixes:
- In spec, memory.copy was missing use of \vconst in the backwards case
- In interpreter & test, memory.init still trapped if segment has been dropped but length parameter was 0.
ioannad pushed a commit to ioannad/exception-handling that referenced this pull request Feb 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants