Skip to content

Conversation

@tlively
Copy link
Member

@tlively tlively commented Sep 12, 2023

Replace i31.new with ref.i31 in the printer, tests, and source code. Continue
parsing i31.new for the time being to allow a graceful transition. Also update
the JS API to reflect the new instruction name.

@tlively tlively requested a review from kripken September 12, 2023 01:26
@tlively
Copy link
Member Author

tlively commented Sep 12, 2023

Current dependencies on/for this PR:

This comment was auto-generated by Graphite.

@codecov
Copy link

codecov bot commented Sep 12, 2023

Codecov Report

Merging #5931 (33fcbfe) into main (9d79632) will decrease coverage by 0.01%.
The diff coverage is 52.94%.

@@            Coverage Diff             @@
##             main    #5931      +/-   ##
==========================================
- Coverage   42.60%   42.60%   -0.01%     
==========================================
  Files         484      484              
  Lines       74815    74823       +8     
  Branches    11919    11920       +1     
==========================================
+ Hits        31872    31875       +3     
- Misses      39746    39751       +5     
  Partials     3197     3197              
Files Changed Coverage Δ
src/passes/RemoveUnusedBrs.cpp 70.30% <ø> (ø)
src/tools/fuzzing/fuzzing.cpp 33.44% <ø> (ø)
src/wasm/wasm-validator.cpp 78.38% <ø> (ø)
src/gen-s-parser.inc 17.88% <50.00%> (+0.05%) ⬆️
src/passes/Print.cpp 66.05% <100.00%> (ø)

... and 2 files with indirect coverage changes

Base automatically changed from ref-i31-internal to main September 13, 2023 15:13
Replace i31.new with ref.i31 in the printer, tests, and source code. Continue
parsing i31.new for the time being to allow a graceful transition. Also update
the JS API to reflect the new instruction name.
@tlively tlively merged commit 2cbe448 into main Sep 14, 2023
@tlively tlively deleted the ref-i31-external branch September 14, 2023 03:02
radekdoulik pushed a commit to dotnet/binaryen that referenced this pull request Jul 12, 2024
Replace i31.new with ref.i31 in the printer, tests, and source code. Continue
parsing i31.new for the time being to allow a graceful transition. Also update
the JS API to reflect the new instruction name.
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