RESOLVED FIXED 210302
Add support for the Wasm i32 sign-extension-ops proposal 
https://bugs.webkit.org/show_bug.cgi?id=210302
Summary Add support for the Wasm i32 sign-extension-ops proposal
Ben Smith
Reported 2020-04-09 13:28:26 PDT
The sign-extension-ops proposal recently became standardized as a phase 5 proposal, and was merged into the spec repo: https://github.com/WebAssembly/spec/pull/1144
Attachments
Patch (12.93 KB, patch)
2020-11-14 13:50 PST, Sergey Rubanov 		no flags
DetailsFormatted DiffDiff
Patch (13.19 KB, patch)
2020-11-14 15:35 PST, Sergey Rubanov 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Patch (13.19 KB, patch)
2020-11-14 15:58 PST, Sergey Rubanov 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Patch (13.19 KB, patch)
2020-11-14 16:14 PST, Sergey Rubanov 		no flags
DetailsFormatted DiffDiff
Patch (12.91 KB, patch)
2020-11-14 19:13 PST, Sergey Rubanov 		no flags
DetailsFormatted DiffDiff
Patch (24.42 KB, patch)
2020-11-15 11:17 PST, Sergey Rubanov 		ysuzuki: review+
ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Patch (24.43 KB, patch)
2020-11-16 08:58 PST, Sergey Rubanov 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Patch (47.89 KB, patch)
2020-11-16 09:30 PST, Sergey Rubanov 		ews-feeder: commit-queue-
DetailsFormatted DiffDiff
Patch (47.89 KB, patch)
2020-11-16 10:00 PST, Sergey Rubanov 		no flags
DetailsFormatted DiffDiff
Show Obsolete (5) View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2020-04-09 15:46:14 PDT
<rdar://problem/61545666>
Keith Miller
Comment 2 2020-04-09 17:10:15 PDT
Yikes! We never did this? Whoops... Thanks for the report!
Sergey Rubanov
Comment 3 2020-11-14 13:50:54 PST
Created attachment 414143 [details] Patch
EWS Watchlist
Comment 4 2020-11-14 13:52:05 PST
This patch modifies one of the wasm.json files. Please ensure that any changes in one have been mirrored to the other. You can find the wasm.json files at "Source/JavaScriptCore/wasm/wasm.json" and "JSTests/wasm/wasm.json".
Sergey Rubanov
Comment 5 2020-11-14 13:53:11 PST
This patch implements only sign extensions to i32 for now. It passes spect tests from https://bugs.webkit.org/show_bug.cgi?id=218940 i62 is WIP
Sergey Rubanov
Comment 6 2020-11-14 15:35:33 PST
Created attachment 414145 [details] Patch
Sergey Rubanov
Comment 7 2020-11-14 15:58:37 PST
Created attachment 414147 [details] Patch
Sergey Rubanov
Comment 8 2020-11-14 16:14:16 PST
Created attachment 414148 [details] Patch
Sergey Rubanov
Comment 9 2020-11-14 19:13:47 PST
Created attachment 414150 [details] Patch
Sergey Rubanov
Comment 10 2020-11-15 11:17:18 PST
Created attachment 414173 [details] Patch
Yusuke Suzuki
Comment 11 2020-11-16 08:43:05 PST
Comment on attachment 414173 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=414173&action=review r=me Nice. > Source/JavaScriptCore/ChangeLog:3 > + Add support for the Wasm sign-extension-ops proposal Can you change the bug title + ChangeLog titles to clarify this implements i32 sign-extension-ops and it is not including i64 ops? And, can you open a bug for i64 sign-extension-ops, and change JSTests/wasm/spec-tests/i64.wast.js's URL to the new one?
Sergey Rubanov
Comment 12 2020-11-16 08:58:29 PST
Created attachment 414239 [details] Patch
Sergey Rubanov
Comment 13 2020-11-16 09:11:47 PST
Bug for i64 sign-extension-ops proposal https://bugs.webkit.org/show_bug.cgi?id=218990
Sergey Rubanov
Comment 14 2020-11-16 09:30:22 PST
Created attachment 414242 [details] Patch
Yusuke Suzuki
Comment 15 2020-11-16 09:34:29 PST
Comment on attachment 414242 [details] Patch OK, for GTK and WPE builds, I ensured that GTK / WPE build work on my Linux (Debian unstable) box, so I think this is build-related stale, and clean-build will fix the issue. And we know that the code in this patch is not changed from r+ patch. I'll put cq+ here.
EWS
Comment 16 2020-11-16 09:35:01 PST
ChangeLog entry in JSTests/ChangeLog contains OOPS!.
Sergey Rubanov
Comment 17 2020-11-16 10:00:16 PST
Created attachment 414245 [details] Patch
EWS
Comment 18 2020-11-17 15:49:25 PST
Committed r269929: <https://trac.webkit.org/changeset/269929> All reviewed patches have been landed. Closing bug and clearing flags on attachment 414245 [details].
Note You need to log in before you can comment on or make changes to this bug.