BridgeJS: expand integer and raw-value enum support by kateinoigakukun · Pull Request #701 · swiftwasm/JavaScriptKit
Navigation Menu
- Notifications You must be signed in to change notification settings
- Fork 69
Conversation
Summary
- add BridgeJS support for fixed-width integer types across code generation, link-time JS glue, and runtime intrinsics
- simplify raw-value enum modeling by reusing
BridgeIntegerTypeinstead of maintaining a separate integer-width mapping - complete the missing optional and 64-bit raw-value enum paths, including correct
biginthandling for JavaScript glue - expand snapshot, runtime, optional, and array coverage for fixed-width integers and raw-value enums, then regenerate checked-in BridgeJS outputs
Testing
mise exec -- ./Utilities/format.swiftmise exec -- swift test --package-path ./Plugins/BridgeJS --filter BridgeJSCodegenTestsmise exec -- swift test --package-path ./Plugins/BridgeJS --filter BridgeJSLinkTestsmise exec -- ./Utilities/bridge-js-generate.shmise exec -- make unittest
kateinoigakukun
changed the title
Align BridgeJS raw-value handling and tests
BridgeJS: expand integer and raw-value enum support
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great 👏🏻 🙌🏻
krodak
deleted the
yt/more-int-width
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters