Skip to content

bump wasmtime 46.0.0#1230

Merged
jacobsteves merged 2 commits into
mainfrom
js.wasmtime-46
Jun 30, 2026
Merged

bump wasmtime 46.0.0#1230
jacobsteves merged 2 commits into
mainfrom
js.wasmtime-46

Conversation

@jacobsteves

@jacobsteves jacobsteves commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

Description of the change

Updates wasmtime. I also had to bump the integration test benchmarks because bytecodealliance/wasmtime#13387 added additional fuel consumption that wasn't previously accounted for.

Why am I making this change?

There was a new major version released

Checklist

  • I've updated the default plugin import namespace and incremented the major version of javy-plugin-api if the QuickJS bytecode has changed.
  • I've updated the relevant CHANGELOG files if necessary. Changes to javy-cli, javy-plugin, and javy-plugin-processing do not require updating CHANGELOG files.
  • I've updated the relevant crate versions if necessary. Versioning policy for library crates
  • I've updated documentation including crate documentation if necessary.

@jeffcharles jeffcharles self-requested a review June 29, 2026 16:54

let (_, _, fuel_consumed) = run(&mut runner, vec![]);
assert_fuel_consumed_within_threshold(18_182, fuel_consumed);
assert_fuel_consumed_within_threshold(18_909, fuel_consumed);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do we have an idea why the increase? Does v46 include any new logic related to fuel calculation?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I believe this should be related to bytecodealliance/wasmtime#13387

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

yeah exactly. I ran a benchmark with those changes disabled and was able to isolate those changes to that issue, heres a benchmark:

   empty.wasm
     wasmtime 45:          18,182
     wasmtime 46 patched:  18,237
     wasmtime 46 main:     18,879 +3.83%

   fib.wasm
     wasmtime 45:          62,358
     wasmtime 46 patched:  62,358
     wasmtime 46 main:     64,513 +3.46%

   text-encoding.wasm
     wasmtime 45:          249,691
     wasmtime 46 patched:  249,691
     wasmtime 46 main:     256,975 +2.92%

@jacobsteves jacobsteves merged commit f660f08 into main Jun 30, 2026
4 checks passed
@jacobsteves jacobsteves deleted the js.wasmtime-46 branch June 30, 2026 21:22
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