WebAssembly: We should only create wrappers for functions that can be exported
Created attachment 312277 [details] Patch
Comment on attachment 312277 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=312277&action=review r=me > Source/JavaScriptCore/ChangeLog:9 > + can actually be exported. It appears to be a ~2.5% speedup on WasmBench. nice > Source/JavaScriptCore/ChangeLog:11 > + This patch also removes most of the old testWasmModuleFunctions api from the jsc CLI. 🎉 Also, why "most" here? It looks like you're removing it all > Tools/Scripts/run-jsc-stress-tests:1204 > + if !$quickMode Do you want to give the specTests/Emscripten the same treatment?
Comment on attachment 312277 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=312277&action=review >> Source/JavaScriptCore/ChangeLog:11 >> + This patch also removes most of the old testWasmModuleFunctions api from the jsc CLI. > > 🎉 > > Also, why "most" here? It looks like you're removing it all The others were not removed but rather updated. >> Tools/Scripts/run-jsc-stress-tests:1204 >> + if !$quickMode > > Do you want to give the specTests/Emscripten the same treatment? Done!
Created attachment 312278 [details] Patch for landing
Created attachment 312279 [details] Patch for landing
Comment on attachment 312277 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=312277&action=review >> Source/JavaScriptCore/ChangeLog:9 >> + can actually be exported. It appears to be a ~2.5% speedup on WasmBench. > > nice Whoops, I should have mentioned that it's only 2.5% on compile times.
Created attachment 312328 [details] Patch for landing
Comment on attachment 312328 [details] Patch for landing Clearing flags on attachment: 312328 Committed r217942: <http://trac.webkit.org/changeset/217942>
All reviewed patches have been landed. Closing bug.