Bug 189785 - Convert op_to_this to store mutable data in a side table
Summary: Convert op_to_this to store mutable data in a side table
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Tadeu Zagallo
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-09-20 06:53 PDT by Tadeu Zagallo
Modified: 2018-10-31 03:38 PDT (History)
6 users (show)

See Also:


Attachments
Patch (93.00 KB, patch)
2018-09-20 06:54 PDT, Tadeu Zagallo
no flags Details | Formatted Diff | Diff
Patch (93.02 KB, patch)
2018-09-20 11:53 PDT, Tadeu Zagallo
no flags Details | Formatted Diff | Diff
Patch (94.51 KB, patch)
2018-09-20 16:09 PDT, Tadeu Zagallo
no flags Details | Formatted Diff | Diff
Patch (95.07 KB, patch)
2018-09-20 16:28 PDT, Tadeu Zagallo
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews102 for mac-sierra (668.99 KB, application/zip)
2018-09-20 17:19 PDT, EWS Watchlist
no flags Details
Patch (95.39 KB, patch)
2018-09-20 17:31 PDT, Tadeu Zagallo
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews102 for mac-sierra (463.48 KB, application/zip)
2018-09-20 18:31 PDT, EWS Watchlist
no flags Details
Archive of layout-test-results from ews112 for mac-sierra (345.42 KB, application/zip)
2018-09-20 18:45 PDT, EWS Watchlist
no flags Details
Patch (95.56 KB, patch)
2018-09-20 18:55 PDT, Tadeu Zagallo
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews102 for mac-sierra (800.75 KB, application/zip)
2018-09-20 19:43 PDT, EWS Watchlist
no flags Details
Archive of layout-test-results from ews115 for mac-sierra (1.31 MB, application/zip)
2018-09-20 20:17 PDT, EWS Watchlist
no flags Details
Archive of layout-test-results from ews125 for ios-simulator-wk2 (62.75 MB, application/zip)
2018-09-20 20:41 PDT, EWS Watchlist
no flags Details
Archive of layout-test-results from ews104 for mac-sierra-wk2 (1.30 MB, application/zip)
2018-09-20 21:25 PDT, EWS Watchlist
no flags Details
Patch (104.41 KB, patch)
2018-09-21 13:24 PDT, Tadeu Zagallo
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews102 for mac-sierra (681.80 KB, application/zip)
2018-09-21 14:43 PDT, EWS Watchlist
no flags Details
Archive of layout-test-results from ews112 for mac-sierra (1.58 MB, application/zip)
2018-09-21 14:57 PDT, EWS Watchlist
no flags Details
Archive of layout-test-results from ews106 for mac-sierra-wk2 (1.45 MB, application/zip)
2018-09-21 15:04 PDT, EWS Watchlist
no flags Details
Patch (106.81 KB, patch)
2018-09-21 15:09 PDT, Tadeu Zagallo
no flags Details | Formatted Diff | Diff
Patch (105.08 KB, patch)
2018-09-21 15:20 PDT, Tadeu Zagallo
no flags Details | Formatted Diff | Diff
Patch (105.21 KB, patch)
2018-09-21 15:38 PDT, Tadeu Zagallo
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews102 for mac-sierra (2.70 MB, application/zip)
2018-09-21 17:52 PDT, EWS Watchlist
no flags Details
Archive of layout-test-results from ews121 for ios-simulator-wk2 (2.45 MB, application/zip)
2018-09-21 18:14 PDT, EWS Watchlist
no flags Details
Archive of layout-test-results from ews106 for mac-sierra-wk2 (3.50 MB, application/zip)
2018-09-21 18:48 PDT, EWS Watchlist
no flags Details
Archive of layout-test-results from ews117 for mac-sierra (4.43 MB, application/zip)
2018-09-21 18:52 PDT, EWS Watchlist
no flags Details
Patch (109.04 KB, patch)
2018-09-24 15:29 PDT, Tadeu Zagallo
ews-watchlist: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from ews203 for win-future (12.71 MB, application/zip)
2018-09-24 21:41 PDT, EWS Watchlist
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Tadeu Zagallo 2018-09-20 06:53:37 PDT
Convert op_to_this as a POC for https://bugs.webkit.org/show_bug.cgi?id=187373
Comment 1 Tadeu Zagallo 2018-09-20 06:54:38 PDT
Created attachment 350195 [details]
Patch
Comment 2 Mark Lam 2018-09-20 11:42:32 PDT
Comment on attachment 350195 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=350195&action=review

> Source/JavaScriptCore/bytecode/OpToThis.h:87
> +    ASSERT(opcodeID == op_to_this);

Make this ASSERT_UNUSED(opcodeID, opcodeID == op_to_this) to fix the release build failures.
Comment 3 Tadeu Zagallo 2018-09-20 11:53:42 PDT
Created attachment 350243 [details]
Patch
Comment 4 Tadeu Zagallo 2018-09-20 16:09:25 PDT
Created attachment 350276 [details]
Patch
Comment 5 Tadeu Zagallo 2018-09-20 16:28:23 PDT
Created attachment 350278 [details]
Patch
Comment 6 EWS Watchlist 2018-09-20 17:19:45 PDT
Comment on attachment 350278 [details]
Patch

Attachment 350278 [details] did not pass mac-ews (mac):
Output: https://webkit-queues.webkit.org/results/9288547

Number of test failures exceeded the failure limit.
Comment 7 EWS Watchlist 2018-09-20 17:19:47 PDT
Created attachment 350284 [details]
Archive of layout-test-results from ews102 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 8 Tadeu Zagallo 2018-09-20 17:31:26 PDT
Created attachment 350287 [details]
Patch
Comment 9 EWS Watchlist 2018-09-20 18:31:05 PDT
Comment on attachment 350287 [details]
Patch

Attachment 350287 [details] did not pass mac-ews (mac):
Output: https://webkit-queues.webkit.org/results/9289532

Number of test failures exceeded the failure limit.
Comment 10 EWS Watchlist 2018-09-20 18:31:07 PDT
Created attachment 350297 [details]
Archive of layout-test-results from ews102 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 11 EWS Watchlist 2018-09-20 18:45:00 PDT
Comment on attachment 350287 [details]
Patch

Attachment 350287 [details] did not pass mac-debug-ews (mac):
Output: https://webkit-queues.webkit.org/results/9289433

Number of test failures exceeded the failure limit.
Comment 12 EWS Watchlist 2018-09-20 18:45:01 PDT
Created attachment 350300 [details]
Archive of layout-test-results from ews112 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews112  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 13 Tadeu Zagallo 2018-09-20 18:55:37 PDT
Created attachment 350301 [details]
Patch
Comment 14 EWS Watchlist 2018-09-20 19:43:42 PDT
Comment on attachment 350301 [details]
Patch

Attachment 350301 [details] did not pass mac-ews (mac):
Output: https://webkit-queues.webkit.org/results/9290439

Number of test failures exceeded the failure limit.
Comment 15 EWS Watchlist 2018-09-20 19:43:44 PDT
Created attachment 350307 [details]
Archive of layout-test-results from ews102 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 16 EWS Watchlist 2018-09-20 20:17:36 PDT
Comment on attachment 350301 [details]
Patch

Attachment 350301 [details] did not pass mac-debug-ews (mac):
Output: https://webkit-queues.webkit.org/results/9290613

Number of test failures exceeded the failure limit.
Comment 17 EWS Watchlist 2018-09-20 20:17:38 PDT
Created attachment 350312 [details]
Archive of layout-test-results from ews115 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews115  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 18 EWS Watchlist 2018-09-20 20:40:59 PDT
Comment on attachment 350301 [details]
Patch

Attachment 350301 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: https://webkit-queues.webkit.org/results/9290680

Number of test failures exceeded the failure limit.
Comment 19 EWS Watchlist 2018-09-20 20:41:02 PDT
Created attachment 350316 [details]
Archive of layout-test-results from ews125 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews125  Port: ios-simulator-wk2  Platform: Mac OS X 10.13.6
Comment 20 EWS Watchlist 2018-09-20 21:25:00 PDT
Comment on attachment 350301 [details]
Patch

Attachment 350301 [details] did not pass mac-wk2-ews (mac-wk2):
Output: https://webkit-queues.webkit.org/results/9291472

Number of test failures exceeded the failure limit.
Comment 21 EWS Watchlist 2018-09-20 21:25:02 PDT
Created attachment 350325 [details]
Archive of layout-test-results from ews104 for mac-sierra-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews104  Port: mac-sierra-wk2  Platform: Mac OS X 10.12.6
Comment 22 Tadeu Zagallo 2018-09-21 13:24:02 PDT
Created attachment 350414 [details]
Patch
Comment 23 EWS Watchlist 2018-09-21 14:43:29 PDT
Comment on attachment 350414 [details]
Patch

Attachment 350414 [details] did not pass mac-ews (mac):
Output: https://webkit-queues.webkit.org/results/9301919

Number of test failures exceeded the failure limit.
Comment 24 EWS Watchlist 2018-09-21 14:43:31 PDT
Created attachment 350432 [details]
Archive of layout-test-results from ews102 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 25 EWS Watchlist 2018-09-21 14:57:14 PDT
Comment on attachment 350414 [details]
Patch

Attachment 350414 [details] did not pass mac-debug-ews (mac):
Output: https://webkit-queues.webkit.org/results/9301759

Number of test failures exceeded the failure limit.
Comment 26 EWS Watchlist 2018-09-21 14:57:16 PDT
Created attachment 350433 [details]
Archive of layout-test-results from ews112 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews112  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 27 EWS Watchlist 2018-09-21 15:04:47 PDT
Comment on attachment 350414 [details]
Patch

Attachment 350414 [details] did not pass mac-wk2-ews (mac-wk2):
Output: https://webkit-queues.webkit.org/results/9302186

Number of test failures exceeded the failure limit.
Comment 28 EWS Watchlist 2018-09-21 15:04:49 PDT
Created attachment 350436 [details]
Archive of layout-test-results from ews106 for mac-sierra-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-sierra-wk2  Platform: Mac OS X 10.12.6
Comment 29 Tadeu Zagallo 2018-09-21 15:09:00 PDT
Created attachment 350437 [details]
Patch
Comment 30 Tadeu Zagallo 2018-09-21 15:20:21 PDT
Created attachment 350440 [details]
Patch
Comment 31 Tadeu Zagallo 2018-09-21 15:38:43 PDT
Created attachment 350445 [details]
Patch
Comment 32 EWS Watchlist 2018-09-21 17:12:40 PDT
Comment on attachment 350445 [details]
Patch

Attachment 350445 [details] did not pass jsc-ews (mac):
Output: https://webkit-queues.webkit.org/results/9304234

New failing tests:
v8-v6/v8-deltablue.js.ftl-eager-no-cjit
wasm.yaml/wasm/js-api/extension-MemoryMode.js.wasm-no-call-ic
microbenchmarks/delta-blue-try-catch.js.ftl-eager-no-cjit
stress/put-direct-index-broken-2.js.dfg-eager-no-cjit-validate
v8-v6/v8-deltablue.js.dfg-eager
stress/v8-deltablue-strict.js.ftl-eager-no-cjit
microbenchmarks/deltablue-for-of.js.dfg-eager-no-cjit-validate
stress/arrow-functions-as-default-parameter-values.js.no-cjit-collect-continuously
microbenchmarks/deltablue-varargs.js.no-cjit-collect-continuously
stress/arrowfunction-lexical-bind-superproperty.js.ftl-eager-no-cjit
stress/parameter-scoping.js.dfg-eager
v8-v6/v8-deltablue.js.ftl-eager
stress/rest-parameter-is-destructuring.js.ftl-eager
stress/super-property-access.js.no-cjit-collect-continuously
stress/reflect-set.js.ftl-eager
v8-v6/v8-crypto.js.ftl-eager
stress/v8-deltablue-strict.js.dfg-eager-no-cjit-validate
microbenchmarks/poly-proto-clear-js-function-allocation-profile.js.dfg-eager-no-cjit-validate
stress/arrow-functions-as-default-parameter-values.js.dfg-eager-no-cjit-validate
stress/put-by-val-slow-put-array-storage.js.dfg-eager-no-cjit-validate
stress/dfg-put-setter-by-id-class.js.dfg-eager-no-cjit-validate
stress/v8-deltablue-strict.js.no-cjit-collect-continuously
v8-v6/v8-crypto.js.dfg-eager-no-cjit-validate
stress/v8-deltablue-strict.js.ftl-eager
stress/arrow-functions-as-default-parameter-values.js.ftl-eager-no-cjit
stress/spread-calling.js.ftl-eager
stress/put-by-val-slow-put-array-storage.js.ftl-eager-no-cjit
microbenchmarks/delta-blue-try-catch.js.dfg-eager
stress/ftl-put-by-id-setter-exception.js.ftl-eager
stress/regress-153486.js.dfg-eager
wasm.yaml/wasm/js-api/extension-MemoryMode.js.wasm-no-tls-context
stress/v8-deltablue-strict.js.dfg-eager
stress/proxy-with-unbalanced-getter-setter.js.no-cjit-collect-continuously
microbenchmarks/delta-blue-try-catch.js.no-cjit-collect-continuously
microbenchmarks/delta-blue-try-catch.js.dfg-eager-no-cjit-validate
v8-v6/v8-deltablue.js.no-cjit-collect-continuously
stress/dfg-put-accessors-by-id.js.ftl-eager-no-cjit
wasm.yaml/wasm/js-api/extension-MemoryMode.js.default-wasm
apiTests
Comment 33 EWS Watchlist 2018-09-21 17:52:34 PDT
Comment on attachment 350445 [details]
Patch

Attachment 350445 [details] did not pass mac-ews (mac):
Output: https://webkit-queues.webkit.org/results/9305080

New failing tests:
imported/w3c/web-platform-tests/WebCryptoAPI/generateKey/failures_AES-CTR.https.any.worker.html
imported/w3c/web-platform-tests/WebCryptoAPI/generateKey/failures_ECDSA.https.any.worker.html
webgl/1.0.2/conformance/uniforms/out-of-bounds-uniform-array-access.html
Comment 34 EWS Watchlist 2018-09-21 17:52:37 PDT
Created attachment 350465 [details]
Archive of layout-test-results from ews102 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 35 EWS Watchlist 2018-09-21 18:14:21 PDT
Comment on attachment 350445 [details]
Patch

Attachment 350445 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: https://webkit-queues.webkit.org/results/9304971

New failing tests:
imported/w3c/web-platform-tests/service-workers/service-worker/interfaces-sw.https.html
Comment 36 EWS Watchlist 2018-09-21 18:14:25 PDT
Created attachment 350466 [details]
Archive of layout-test-results from ews121 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews121  Port: ios-simulator-wk2  Platform: Mac OS X 10.13.6
Comment 37 EWS Watchlist 2018-09-21 18:48:18 PDT
Comment on attachment 350445 [details]
Patch

Attachment 350445 [details] did not pass mac-wk2-ews (mac-wk2):
Output: https://webkit-queues.webkit.org/results/9305648

New failing tests:
imported/w3c/web-platform-tests/WebCryptoAPI/generateKey/failures_HMAC.https.any.worker.html
imported/w3c/web-platform-tests/service-workers/service-worker/interfaces-sw.https.html
Comment 38 EWS Watchlist 2018-09-21 18:48:22 PDT
Created attachment 350472 [details]
Archive of layout-test-results from ews106 for mac-sierra-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-sierra-wk2  Platform: Mac OS X 10.12.6
Comment 39 EWS Watchlist 2018-09-21 18:52:23 PDT
Comment on attachment 350445 [details]
Patch

Attachment 350445 [details] did not pass mac-debug-ews (mac):
Output: https://webkit-queues.webkit.org/results/9305405

New failing tests:
imported/w3c/web-platform-tests/WebCryptoAPI/generateKey/successes_AES-GCM.https.any.worker.html
imported/w3c/web-platform-tests/WebCryptoAPI/generateKey/failures_ECDH.https.any.worker.html
Comment 40 EWS Watchlist 2018-09-21 18:52:27 PDT
Created attachment 350473 [details]
Archive of layout-test-results from ews117 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews117  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 41 Tadeu Zagallo 2018-09-24 15:29:00 PDT
Created attachment 350699 [details]
Patch
Comment 42 EWS Watchlist 2018-09-24 21:41:02 PDT
Comment on attachment 350699 [details]
Patch

Attachment 350699 [details] did not pass win-ews (win):
Output: https://webkit-queues.webkit.org/results/9339035

New failing tests:
fast/workers/worker-exception-during-navigation.html
Comment 43 EWS Watchlist 2018-09-24 21:41:17 PDT
Created attachment 350737 [details]
Archive of layout-test-results from ews203 for win-future

The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews203  Port: win-future  Platform: CYGWIN_NT-6.1-2.9.0-0.318-5-3-x86_64-64bit
Comment 44 Tadeu Zagallo 2018-10-31 03:38:43 PDT
The bytecode rewrite has landed in r237547 <https://trac.webkit.org/r237547>