Skip to content

feat: update binaryen to v130#3026

Merged
MaxGraey merged 4 commits into
AssemblyScript:mainfrom
MaxGraey:upd-binaryen
Jun 11, 2026
Merged

feat: update binaryen to v130#3026
MaxGraey merged 4 commits into
AssemblyScript:mainfrom
MaxGraey:upd-binaryen

Conversation

@MaxGraey

@MaxGraey MaxGraey commented Jun 9, 2026

Copy link
Copy Markdown
Member

Also:

  • Add a function table before indirect call analysis (required by new binaryen)
  • Add a two new api methods for the wide arith integers proposal
  • I've read the contributing guidelines
  • I've added my name and email to the NOTICE file

@MaxGraey MaxGraey changed the title Update binaryen to v130 feat: update binaryen to v130 Jun 9, 2026
@MaxGraey MaxGraey merged commit 18b17a7 into AssemblyScript:main Jun 11, 2026
14 checks passed
@MaxGraey MaxGraey deleted the upd-binaryen branch June 11, 2026 16:00
BlobMaster41 added a commit to dacely-cloud/toilscript that referenced this pull request Jun 19, 2026
…Script#3026)

Brings in the single upstream commit upgrading binaryen 129 -> 130
(130.0.0-nightly.20260609). Most of it auto-merged.

src/module.ts gains the v130 C-API bindings: wide-int add/sub and mul
ops (i64.add128/sub128, i64.mul_wide_s/u), ensureFunctionTable(), and
the renamed relaxed-SIMD / ref.as constants (RelaxedLaneselect*,
AnyConvertExtern / ExternConvertAny). src/compiler.ts adopts upstream's
lazy table declaration (declare the default table on first indirect use
so binaryen's effect analysis sees it), alongside our closures path.

Conflict: tests/compiler/assert-nonnull.release.wat, a generated
fixture. Resolved by regenerating from our compiler rather than
hand-merging. Our fork's abort wrapper (env.abort -> __abort_impl) is
preserved; v130's improved effect analysis now drops the dead
call_indirect in unreachable blocks and the resulting unused empty
table, matching upstream's intent for this fixture.

Regenerated and verified the auto-merged feature/relaxed-simd and
ternary.release fixtures against our compiler output. Full npm test
suite green (parser, compiler, browser, toilconfig, transform, cli,
json, data). Pre-existing 'npm run check' lint failures are in
untouched fork files and unrelated to this merge.
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.

1 participant