Bug 125382

Summary: [Win][64-bit] Hitting breakpoint assembler instruction in callToJavaScript.
Product: WebKit Reporter: peavo
Component: JavaScriptCoreAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, commit-queue, msaboff
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 124450    
Attachments:
Description Flags
Patch none

Description peavo 2013-12-07 01:48:07 PST
When running WinCairo 64-bit with JIT enabled, I'm currently hitting breakpoint instructions in the callToJavaScript function.
Removing these, the results from the run-javascriptcore-tests (including stress tests, and api tests) are the same as for WinCairo 32-bit.
Also "normal" browsing seems to work fine. 

Results from run-javascriptcore-tests:

7654/7654 (failed 12)

** The following JSC stress test failures have been introduced:
        mozilla-tests.yaml/ecma_3/Date/15.9.5.6.js.mozilla
        mozilla-tests.yaml/ecma_3/Date/15.9.5.6.js.mozilla-llint
        mozilla-tests.yaml/ecma_3/Date/15.9.5.6.js.mozilla-baseline
        mozilla-tests.yaml/ecma_3/Date/15.9.5.6.js.mozilla-dfg-eager-no-cjit-validate-phases
        jsc-layout-tests.yaml/js/script-tests/math.js.layout
        jsc-layout-tests.yaml/js/script-tests/math.js.layout-no-llint
        jsc-layout-tests.yaml/js/script-tests/math.js.layout-no-cjit
        jsc-layout-tests.yaml/js/script-tests/math.js.layout-dfg-eager-no-cjit
        jsc-layout-tests.yaml/js/script-tests/sort-stability.js.layout
        jsc-layout-tests.yaml/js/script-tests/sort-stability.js.layout-no-llint
        jsc-layout-tests.yaml/js/script-tests/sort-stability.js.layout-no-cjit
        jsc-layout-tests.yaml/js/script-tests/sort-stability.js.layout-dfg-eager-no-cjit

Results for JSC stress tests:
    12 failures found.
Comment 1 peavo 2013-12-07 01:53:51 PST
Created attachment 218654 [details]
Patch
Comment 2 Michael Saboff 2013-12-07 08:03:50 PST
Comment on attachment 218654 [details]
Patch

r=me.  Thanks for testing this.  Good to see that it works.
Comment 3 WebKit Commit Bot 2013-12-07 08:30:44 PST
Comment on attachment 218654 [details]
Patch

Clearing flags on attachment: 218654

Committed r160272: <http://trac.webkit.org/changeset/160272>
Comment 4 WebKit Commit Bot 2013-12-07 08:30:46 PST
All reviewed patches have been landed.  Closing bug.