Bug 185989

Summary: JavaScriptCore: Disable 32-bit JIT on Windows
Product: WebKit Reporter: Michael Saboff <msaboff>
Component: JavaScriptCoreAssignee: Michael Saboff <msaboff>
Status: RESOLVED FIXED    
Severity: Normal CC: benjamin, cdumez, cmarcelo, commit-queue, dbates, ews-watchlist, mark.lam, saam, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 186061, 186153, 186477    
Bug Blocks:    
Attachments:
Description Flags
Patch
none
Updated Patch
none
Updated Patch
mark.lam: review+, msaboff: commit-queue-
Archive of layout-test-results from ews201 for win-future none

Description Michael Saboff 2018-05-25 12:58:31 PDT
This code is not being maintained. For some 32 bit apps, the JIT doesn't provide a performance benefit.  Disabling the JIT will also save memory.
Comment 1 Michael Saboff 2018-05-25 12:58:50 PDT
<rdar://problem/40343776>
Comment 2 Michael Saboff 2018-05-25 13:03:45 PDT
Created attachment 341312 [details]
Patch
Comment 3 WebKit Commit Bot 2018-05-25 16:26:48 PDT
Comment on attachment 341312 [details]
Patch

Clearing flags on attachment: 341312

Committed r232212: <https://trac.webkit.org/changeset/232212>
Comment 4 WebKit Commit Bot 2018-05-25 16:26:50 PDT
All reviewed patches have been landed.  Closing bug.
Comment 5 Michael Saboff 2018-05-29 14:47:26 PDT
This change broke the Win32 build.
Comment 6 Michael Saboff 2018-06-01 16:31:03 PDT
Created attachment 341801 [details]
Updated Patch
Comment 7 EWS Watchlist 2018-06-01 16:33:01 PDT
Attachment 341801 [details] did not pass style-queue:


ERROR: Source/WTF/ChangeLog:1:  ChangeLog entry has no bug number  [changelog/bugnumber] [5]
Total errors found: 1 in 8 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 8 Michael Saboff 2018-06-01 16:35:53 PDT
Created attachment 341802 [details]
Updated Patch
Comment 9 Mark Lam 2018-06-01 16:46:01 PDT
Comment on attachment 341802 [details]
Updated Patch

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

r=me if the EWS bots are happy.

> Source/JavaScriptCore/ChangeLog:8
> +        Made various minor fixes.

I think you should mention that these fixes are so that the CLOOP can work when COMPUTED_GOTOs is not supported.

> Source/JavaScriptCore/ChangeLog:13
> +        * llint/LowLevelInterpreter.cpp: Changed the definition of OFFLINE_ASM_GLOBAL_LABEL to not
> +        have a case label these aren't opcodes.

Did you mean "... case label because these aren't opcodes"?
Comment 10 EWS Watchlist 2018-06-01 20:24:26 PDT
Comment on attachment 341802 [details]
Updated Patch

Attachment 341802 [details] did not pass win-ews (win):
Output: http://webkit-queues.webkit.org/results/7938086

New failing tests:
js/slow-stress/emscripten-memops.html
workers/bomb.html
js/dom/line-column-numbers.html
js/regexp-zero-length-alternatives.html
fast/workers/worker-document-leak.html
accessibility/accessibility-node-memory-management.html
storage/indexeddb/modern/leak-1.html
Comment 11 EWS Watchlist 2018-06-01 20:24:37 PDT
Created attachment 341822 [details]
Archive of layout-test-results from ews201 for win-future

The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews201  Port: win-future  Platform: CYGWIN_NT-6.1-2.9.0-0.318-5-3-x86_64-64bit
Comment 12 Michael Saboff 2018-06-10 22:33:26 PDT
(In reply to Build Bot from comment #10)
> Comment on attachment 341802 [details]
> Updated Patch
> 
> Attachment 341802 [details] did not pass win-ews (win):
> Output: http://webkit-queues.webkit.org/results/7938086
> 
> New failing tests:
...
> js/regexp-zero-length-alternatives.html
...

The failure of js/regexp-zero-length-alternatives.html is being tracked in https://bugs.webkit.org/show_bug.cgi?id=186477
Comment 13 Michael Saboff 2018-06-11 09:39:51 PDT
Committed r232719: <https://trac.webkit.org/changeset/232719>