Bug 125382 - [Win][64-bit] Hitting breakpoint assembler instruction in callToJavaScript.
Summary: [Win][64-bit] Hitting breakpoint assembler instruction in callToJavaScript.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 124450
  Show dependency treegraph
 
Reported: 2013-12-07 01:48 PST by peavo
Modified: 2013-12-07 08:30 PST (History)
3 users (show)

See Also:


Attachments
Patch (2.25 KB, patch)
2013-12-07 01:53 PST, peavo
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.