WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
160914
Add WASM I32 simple operators.
https://bugs.webkit.org/show_bug.cgi?id=160914
Summary
Add WASM I32 simple operators.
Keith Miller
Reported
2016-08-16 15:10:10 PDT
Add WASM I32 simple operators.
Attachments
Patch
(4.82 KB, patch)
2016-08-16 16:12 PDT
,
Keith Miller
no flags
Details
Formatted Diff
Diff
Patch
(4.78 KB, patch)
2016-08-16 16:15 PDT
,
Keith Miller
benjamin
: review+
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Keith Miller
Comment 1
2016-08-16 16:12:58 PDT
Created
attachment 286225
[details]
Patch
Keith Miller
Comment 2
2016-08-16 16:15:37 PDT
Created
attachment 286227
[details]
Patch
WebKit Commit Bot
Comment 3
2016-08-16 16:16:48 PDT
Attachment 286227
[details]
did not pass style-queue: ERROR: Source/JavaScriptCore/wasm/WASMOps.h:49: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/JavaScriptCore/wasm/WASMOps.h:58: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/JavaScriptCore/wasm/WASMOps.h:59: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/JavaScriptCore/wasm/WASMB3IRGenerator.cpp:48: Non-label code inside switch statements should be indented. [whitespace/indent] [4] Total errors found: 4 in 4 files If any of these errors are false positives, please file a bug against check-webkit-style.
Keith Miller
Comment 4
2016-08-16 16:34:59 PDT
Landed in
https://trac.webkit.org/changeset/204539
.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug