Skip to content

Conversation

@cfallin
Copy link
Member

@cfallin cfallin commented Jul 2, 2025

This updates weval as part of an effort to update to the latest SpiderMonkey in wevaled-SpiderMonkey builds and latest wasi-sdk. In particular:

  • Updates to Wizer 9.0, which supports reftypes natively (emitted by the latest LLVM in recent wasi-sdk).
  • Adds more detailed log messages in certain cases, helpful for tracking down reasons for non-specialized code due to changes in code structure or LLVM optimizations.
  • Adds an intrinsic (assert_specialized) that I found useful during said debugging.

@cfallin cfallin requested a review from fitzgen July 2, 2025 23:03
This updates weval as part of an effort to update to the latest
SpiderMonkey in wevaled-SpiderMonkey builds and latest wasi-sdk. In
particular:

- Updates to Wizer 9.0, which supports reftypes natively (emitted by
  the latest LLVM in recent wasi-sdk).
- Adds more detailed log messages in certain cases, helpful for
  tracking down reasons for non-specialized code due to changes in
  code structure or LLVM optimizations.
- Adds an intrinsic (`assert_specialized`) that I found useful during
  said debugging.
@cfallin cfallin force-pushed the cfallin/update-wizer-and-add-debug-helpers branch from deb59b0 to 5488548 Compare July 2, 2025 23:04
Copy link
Member

@sunfishcode sunfishcode left a comment

Choose a reason for hiding this comment

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

For what it's worth, clang isn't using actual reference types; it's just using the overlong call-indirect encoding which is part of the reference-types feature.

@cfallin cfallin merged commit 83ad01b into main Jul 2, 2025
12 checks passed
@cfallin cfallin deleted the cfallin/update-wizer-and-add-debug-helpers branch July 2, 2025 23:17
@cfallin cfallin mentioned this pull request Jul 2, 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.

3 participants