-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[JSC] Remove Wasm::PinnedRegisterInfo #8900
[JSC] Remove Wasm::PinnedRegisterInfo #8900
Conversation
EWS run on previous version of this PR (hash 674bb03) |
674bb03
to
d294d0c
Compare
EWS run on previous version of this PR (hash d294d0c) |
d294d0c
to
99bf8ee
Compare
EWS run on previous version of this PR (hash 99bf8ee) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
r=me
99bf8ee
to
a3d215e
Compare
https://bugs.webkit.org/show_bug.cgi?id=250920 rdar://104496000 Reviewed by Mark Lam. This patch removes Wasm::PinnedRegisterInfo since we no longer have TLS version. So, these register information can be defined at compile time. We should just define them in GPRInfo, easy to read what is specified in each architecture. * Source/JavaScriptCore/jit/AssemblyHelpers.cpp: (JSC::AssemblyHelpers::loadWasmContextInstance): (JSC::AssemblyHelpers::storeWasmContextInstance): * Source/JavaScriptCore/jit/GPRInfo.h: * Source/JavaScriptCore/jit/RegisterSet.cpp: (JSC::RegisterSetBuilder::wasmPinnedRegisters): * Source/JavaScriptCore/jit/RegisterSet.h: * Source/JavaScriptCore/wasm/WasmAirIRGeneratorBase.h: (JSC::Wasm::ExpressionType>::AirIRGeneratorBase): (JSC::Wasm::ExpressionType>::restoreWebAssemblyGlobalState): (JSC::Wasm::ExpressionType>::addCall): (JSC::Wasm::ExpressionType>::emitIndirectCall): * Source/JavaScriptCore/wasm/WasmB3IRGenerator.cpp: (JSC::Wasm::B3IRGenerator::B3IRGenerator): (JSC::Wasm::B3IRGenerator::restoreWebAssemblyGlobalState): (JSC::Wasm::B3IRGenerator::emitIndirectCall): (JSC::Wasm::B3IRGenerator::addCall): * Source/JavaScriptCore/wasm/WasmBinding.cpp: (JSC::Wasm::wasmToWasm): * Source/JavaScriptCore/wasm/WasmMemoryInformation.cpp: (JSC::Wasm::PinnedRegisterInfo::get): Deleted. (JSC::Wasm::PinnedRegisterInfo::PinnedRegisterInfo): Deleted. * Source/JavaScriptCore/wasm/WasmMemoryInformation.h: (JSC::Wasm::PinnedRegisterInfo::toSave const): Deleted. * Source/JavaScriptCore/wasm/WasmOperations.cpp: (JSC::Wasm::JSC_DEFINE_JIT_OPERATION): * Source/JavaScriptCore/wasm/js/JSToWasm.cpp: (JSC::Wasm::marshallJSResult): (JSC::Wasm::createJSToWasmWrapper): * Source/JavaScriptCore/wasm/js/WasmToJS.cpp: (JSC::Wasm::materializeImportJSCell): (JSC::Wasm::wasmToJS): * Source/JavaScriptCore/wasm/js/WebAssemblyFunction.cpp: (JSC::WebAssemblyFunction::calleeSaves const): (JSC::WebAssemblyFunction::jsCallEntrypointSlow): Canonical link: https://commits.webkit.org/259161@main
a3d215e
to
4dae653
Compare
Committed 259161@main (4dae653): https://commits.webkit.org/259161@main Reviewed commits have been landed. Closing PR #8900 and removing active labels. |
4dae653
a3d215e
π macπ π§ͺ winπ§ͺ ios-wk2π§ͺ api-macπ§ͺ gtk-wk2π§ͺ api-iosπ§ͺ mac-wk1π§ͺ api-gtkπ π§ͺ jscπ tvπ§ͺ mac-wk2π π§ͺ jsc-arm64π§ͺ mac-AS-debug-wk2π§ͺ jsc-armv7-testsπ§ͺ mac-wk2-stressπ§ͺ jsc-mips-tests