Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LibWasm+AK: Sync Wasm changes from ladybird #24682

Merged
merged 9 commits into from
Jul 9, 2024

Conversation

alimpfard
Copy link
Member

(cherry-picked)

(cherry picked from commit fce8ed15630a4969be7c9761b9b7d3cef0530cc6)
A `Uint8Array` can now be passed in the Wasm testjs files to be
transmuted into a Wasm value.

(cherry picked from commit 6493acf2f46f6e4522a0f1f01e12d0e48347562e)
NaN bit patterns are now (hopefully) preserved. `static_cast` does not
preserve the bit pattern of a given NaN, so ideally we'd use some other
sort of cast and avoid `static_cast` altogether, but that's a large
change for this commit. For now, this fixes the issues found in spec
tests.

(cherry picked from commit c882498d4450c4c2e46d77a8ab36afc4eb412c00)
(cherry picked from commit c103001f167d39fd68c4f04649332f76b7dd0b2e)
According to the spec, -0.0 < 0.0.

(cherry picked from commit 31c7e98a4a46c2d0ef93c5fca47d64d05b96449f)
As far as I know, they're not in the spec and don't serve any purposes
in the internals of LibWasm.

(cherry picked from commit 5382fbb6171555264e29872029330e1373b39671)
Simply returns how many bytes can be read from the stream.

(cherry picked from commit aee2f25929157a4e6c7f391584c4839a40beadc8)
(cherry picked from commit afd8d90f3237e0cffaabb05becbff16f8c8fdd25)
@alimpfard alimpfard requested a review from timschumi as a code owner July 9, 2024 22:41
@github-actions github-actions bot added the 👀 pr-needs-review PR needs review from a maintainer or community member label Jul 9, 2024
@alimpfard alimpfard merged commit 5e8dba0 into SerenityOS:master Jul 9, 2024
12 checks passed
@alimpfard alimpfard deleted the wasm-sync branch July 9, 2024 23:10
@github-actions github-actions bot removed the 👀 pr-needs-review PR needs review from a maintainer or community member label Jul 9, 2024
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.

None yet

2 participants