Summary: | The relationship between abstract values and structure transition watchpoints should be rationalized | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Filip Pizlo <fpizlo> | ||||||||
Component: | JavaScriptCore | Assignee: | Filip Pizlo <fpizlo> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | barraclough, fpizlo, ggaren, mhahnenberg, msaboff, oliver, webkit.review.bot | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | All | ||||||||||
OS: | All | ||||||||||
Bug Depends on: | 94201, 94408, 94412 | ||||||||||
Bug Blocks: | 91933 | ||||||||||
Attachments: |
|
Description
Filip Pizlo
2012-08-16 01:18:03 PDT
Created attachment 158746 [details]
work in progress
Created attachment 158976 [details]
almost there
It looks like it's working but I still need to do more perf tests.
Comment on attachment 158976 [details] almost there Attachment 158976 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/13511820 Comment on attachment 158976 [details] almost there Attachment 158976 [details] did not pass qt-wk2-ews (qt): Output: http://queues.webkit.org/results/13520265 (In reply to comment #4) > (From update of attachment 158976 [details]) > Attachment 158976 [details] did not pass qt-wk2-ews (qt): > Output: http://queues.webkit.org/results/13520265 Looks like I still have to make 32-bit changes. Will do so soon; first I want to test that the 64-bit part is at least perf-neutral. Created attachment 159026 [details]
the patch
Attachment 159026 [details] did not pass style-queue:
Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/ChangeLog', u'Source..." exit_code: 1
Source/JavaScriptCore/dfg/DFGAbstractValue.h:247: Should have only a single space after a punctuation in a comment. [whitespace/comments] [5]
Total errors found: 1 in 18 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 159026 [details]
the patch
r=me
Landed in http://trac.webkit.org/changeset/125999 |