Skip to content

Enable Wasm BBQ JIT on Windows#31185

Merged
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
iangrunert:ig/enable-wasm-bbq-win
Jul 25, 2024
Merged

Enable Wasm BBQ JIT on Windows#31185
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
iangrunert:ig/enable-wasm-bbq-win

Conversation

@iangrunert
Copy link
Contributor

@iangrunert iangrunert commented Jul 24, 2024

11682f9

Enable Wasm BBQ JIT on Windows
https://bugs.webkit.org/show_bug.cgi?id=277037

Reviewed by Yusuke Suzuki.

Inside tryAllocateZeroedVirtualPages we were calling
OSAllocator::reserveAndCommit which crashes if it fails to allocate,
instead of tryReserveAndCommit which returns a null pointer. This caused
a number of memory-related wasm test failures.

* JSTests/wasm/v8/shared-memory-gc-stress.js:
* Source/JavaScriptCore/llint/WebAssembly.asm:
* Source/JavaScriptCore/runtime/Options.cpp:
(JSC::Options::notifyOptionsChanged):
* Source/JavaScriptCore/wasm/WasmBBQJIT.cpp:
(JSC::Wasm::BBQJITImpl::Location::none):
* Source/WTF/wtf/Gigacage.cpp:
(Gigacage::tryAllocateZeroedVirtualPages):

Canonical link: https://commits.webkit.org/281324@main

cde0f12

Misc iOS, visionOS, tvOS & watchOS macOS Linux Windows
✅ 🧪 style ✅ 🛠 ios ✅ 🛠 mac ✅ 🛠 wpe ✅ 🛠 wincairo
✅ 🛠 ios-sim ✅ 🛠 mac-AS-debug 🧪 wpe-wk2 ✅ 🧪 wincairo-tests
✅ 🧪 webkitperl ✅ 🧪 ios-wk2 ✅ 🧪 api-mac ✅ 🧪 api-wpe
✅ 🧪 ios-wk2-wpt ✅ 🧪 mac-wk1 ✅ 🛠 wpe-cairo
✅ 🛠 🧪 jsc ✅ 🧪 api-ios ✅ 🧪 mac-wk2 ✅ 🛠 gtk
✅ 🛠 🧪 jsc-arm64 ✅ 🛠 vision 🧪 mac-AS-debug-wk2 🧪 gtk-wk2
✅ 🛠 vision-sim ✅ 🧪 mac-wk2-stress ✅ 🧪 api-gtk
✅ 🧪 vision-wk2 ✅ 🛠 jsc-armv7
✅ 🛠 🧪 unsafe-merge ✅ 🛠 tv ✅ 🧪 jsc-armv7-tests
🛠 tv-sim
✅ 🛠 watch
✅ 🛠 watch-sim

@iangrunert iangrunert requested a review from a team as a code owner July 24, 2024 21:27
@iangrunert iangrunert self-assigned this Jul 24, 2024
@iangrunert iangrunert force-pushed the ig/enable-wasm-bbq-win branch from 2e0383c to cde0f12 Compare July 24, 2024 21:32
Copy link
Member

@Constellation Constellation left a comment

Choose a reason for hiding this comment

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

r=me

@Constellation Constellation added the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Jul 25, 2024
https://bugs.webkit.org/show_bug.cgi?id=277037

Reviewed by Yusuke Suzuki.

Inside tryAllocateZeroedVirtualPages we were calling
OSAllocator::reserveAndCommit which crashes if it fails to allocate,
instead of tryReserveAndCommit which returns a null pointer. This caused
a number of memory-related wasm test failures.

* JSTests/wasm/v8/shared-memory-gc-stress.js:
* Source/JavaScriptCore/llint/WebAssembly.asm:
* Source/JavaScriptCore/runtime/Options.cpp:
(JSC::Options::notifyOptionsChanged):
* Source/JavaScriptCore/wasm/WasmBBQJIT.cpp:
(JSC::Wasm::BBQJITImpl::Location::none):
* Source/WTF/wtf/Gigacage.cpp:
(Gigacage::tryAllocateZeroedVirtualPages):

Canonical link: https://commits.webkit.org/281324@main
@webkit-commit-queue
Copy link
Collaborator

Committed 281324@main (11682f9): https://commits.webkit.org/281324@main

Reviewed commits have been landed. Closing PR #31185 and removing active labels.

@webkit-commit-queue webkit-commit-queue merged commit 11682f9 into WebKit:main Jul 25, 2024
@webkit-commit-queue webkit-commit-queue removed the unsafe-merge-queue Applied to send a pull request to merge-queue, but skip building and testing label Jul 25, 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.

4 participants