Skip to content

Conversation

sunfishcode
Copy link
Member

This follows up on a comment in #96 by adding comments for the remaining AST nodes. I also updated a few other nearby AST node comments.

@sunfishcode
Copy link
Member Author

This is just comments; anyone want to lgtm? :-)

@rossberg
Copy link
Member

rossberg commented Oct 8, 2015

Hm, I thought I did already. LGTM

@sunfishcode
Copy link
Member Author

Thanks!

sunfishcode added a commit that referenced this pull request Oct 8, 2015
Comment LoadExtend and StoreTrunc and update comments for other nodes.
@sunfishcode sunfishcode merged commit 58862d8 into master Oct 8, 2015
@sunfishcode sunfishcode deleted the update-comments branch October 8, 2015 15:38
eqrion pushed a commit to eqrion/wasm-spec that referenced this pull request Jul 18, 2019
Also fix some typos from previous CL (see macros.def)
alexcrichton pushed a commit to alexcrichton/spec that referenced this pull request Nov 18, 2019
A poll was held at CG meeting today, the consensus was for Option 3 of
WebAssembly/simd#101.

This means we have all the f64x2 instructions, and some of the more
common i64x2 that have been benchmarked. This change reflects that
option in the proposal text, binary text, and implementation status.

Fixes WebAssembly#101.
Connicpu pushed a commit to Connicpu/wasm-spec that referenced this pull request May 11, 2020
We decided to revert this from i64 to i32 in Nov 28 CG meeting, but the
spec was not updated accordingly.
dhil added a commit to dhil/webassembly-spec that referenced this pull request Dec 3, 2024
This patch brings the encoding and decoding of tag imports in sync
with the exception handling proposal.

Resolves WebAssembly#101.
rossberg pushed a commit that referenced this pull request Jan 20, 2025
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.

2 participants