Use standard GC encodings by default

[tlively]

The legacy encodings remain available for now by defining
USE_LEGACY_GC_ENCODINGS at build time.

[tlively]

• Remove legacy GC encodings #5874
• Use standard GC encodings by default #5873 👈
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tlively and the rest of your teammates on Graphite

[tlively]

This is meant to be landed immediately after the GC proposal goes to phase 4. A vote to promote it to phase 4 is currently scheduled for September 12.

[codecov]

Codecov Report

Merging #5873 (5878f44) into main (616c08b) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #5873      +/-   ##
==========================================
- Coverage   42.46%   42.46%   -0.01%     
==========================================
  Files         482      482              
  Lines       74727    74726       -1     
  Branches    11929    11932       +3     
==========================================
- Hits        31732    31729       -3     
  Misses      39790    39790              
- Partials     3205     3207       +2     

Files Changed	Coverage Δ
src/wasm-binary.h	83.60% <ø> (ø)

... and 5 files with indirect coverage changes

[bashor]

Please also make a new release after the merge.

[tlively]

@bashor, we will make a release by the end of the week, if that's ok for you. It would be good to get as many text format cleanups into the release as possible.

[bashor]

@tlively, it would be nice to have a release as soon as possible since binaryen is one of the dependencies we need to update. We don't use binaryen's text format support.

[bashor]

@tlively any news?

[tlively]

I've just created the release now. CI is currently working on producing the build artifacts for it.

[bashor]

Great! Thanks a lot!