Bug 97288

Summary: JSC should infer when indexed storage is contiguous, and optimize for it
Product: WebKit Reporter: Filip Pizlo <fpizlo>
Component: JavaScriptCoreAssignee: Filip Pizlo <fpizlo>
Status: RESOLVED FIXED    
Severity: Normal CC: barraclough, benjamin, ggaren, gustavo, gyuyoung.kim, mark.lam, mhahnenberg, msaboff, oliver, ossy, philn, rakuco, webkit.review.bot, xan.lopez
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on: 97535, 97537, 98236, 98857, 98859    
Bug Blocks: 98605, 98606, 98608    
Attachments:
Description Flags
it begins
none
it has more things
none
patching is done
none
dfg stuff has begun
none
more
none
almost there!
none
fixing things
none
it compiles!
none
sunspider runs
none
it sort of works
none
new approach: holy contiguous
none
I think it's done (holy contiguous)
none
it compiles (holy contiguous)
none
it works on 64-bit, starting to make it work on 32-bit
none
the patch, possibly
none
the patch
ggaren: review-, webkit-ews: commit-queue-
the patch
webkit-ews: commit-queue-
the patch
gyuyoung.kim: commit-queue-
the patch
webkit-ews: commit-queue-
the patch
buildbot: commit-queue-
the patch
mhahnenberg: review+, buildbot: commit-queue-
patch for landing none

Description Filip Pizlo 2012-09-20 20:47:27 PDT 
Monster work in progress patch coming soon to an attachment near you.
Comment 1 Filip Pizlo 2012-09-20 20:51:45 PDT 
Created attachment 165040 [details]
it begins
Comment 2 Filip Pizlo 2012-09-23 16:41:55 PDT 
Created attachment 165296 [details]
it has more things

Patchable get_by_val's in the baseline JIT.  They're no fun.
Comment 3 Filip Pizlo 2012-09-23 20:15:35 PDT 
Created attachment 165304 [details]
patching is done

get_by_val/put_by_val patching is done.  still need to pull this through the DFG.
Comment 4 Filip Pizlo 2012-09-25 00:37:37 PDT 
Created attachment 165542 [details]
dfg stuff has begun
Comment 5 Filip Pizlo 2012-09-26 01:20:27 PDT 
Created attachment 165747 [details]
more

Wrote a bunch of the DFG code.  Not done yet.
Comment 6 Filip Pizlo 2012-09-26 11:32:34 PDT 
Created attachment 165838 [details]
almost there!

Code is written but it hasn't been compiled or tested.

Oh, and I haven't done anything about 32-bit yet.
Comment 7 Filip Pizlo 2012-09-26 13:25:27 PDT 
Created attachment 165861 [details]
fixing things
Comment 8 Filip Pizlo 2012-09-27 00:17:50 PDT 
Created attachment 165945 [details]
it compiles!
Comment 9 Filip Pizlo 2012-09-29 17:56:29 PDT 
Created attachment 166374 [details]
sunspider runs

I don't know how fast it runs, but at least it runs.  Still lots more work to do before this patch is complete.
Comment 10 Filip Pizlo 2012-09-29 23:14:27 PDT 
Created attachment 166379 [details]
it sort of works

I'm sort of reconsidering whether my idea of ridding the world of holes was the right approach.  Although this patch is already a speed-up, it's looking like it would be an even clearer win if I did have some support for holes in contiguous arrays.
Comment 11 Filip Pizlo 2012-09-30 18:07:53 PDT 
Created attachment 166395 [details]
new approach: holy contiguous

Note yet done.
Comment 12 Filip Pizlo 2012-09-30 18:34:55 PDT 
Created attachment 166397 [details]
I think it's done (holy contiguous)

Still haven't compiled it yet.  What could go wrong.
Comment 13 Filip Pizlo 2012-09-30 20:24:06 PDT 
Created attachment 166401 [details]
it compiles (holy contiguous)

Don't know if it does things yet.
Comment 14 Filip Pizlo 2012-10-04 19:46:40 PDT 
Created attachment 167234 [details]
it works on 64-bit, starting to make it work on 32-bit
Comment 15 Filip Pizlo 2012-10-04 21:56:07 PDT 
Created attachment 167244 [details]
the patch, possibly

I wrote the 32-bit code but I'm not sure if it works, yet.
Comment 16 Filip Pizlo 2012-10-06 15:32:23 PDT 
Created attachment 167462 [details]
the patch

It's ready!
Comment 17 WebKit Review Bot 2012-10-06 15:35:49 PDT 
Attachment 167462 [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/runtime/JSObject.h:695:  The parameter name "butterfly" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSObject.h:764:  The parameter name "globalData" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/dfg/DFGOperations.cpp:609:  When wrapping a line, only indent 4 spaces.  [whitespace/indent] [3]
Source/JavaScriptCore/runtime/JSArray.cpp:1118:  Should have only a single space after a punctuation in a comment.  [whitespace/comments] [5]
Source/JavaScriptCore/jit/JITPropertyAccess.cpp:1430:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/JavaScriptCore/jit/JITPropertyAccess.cpp:1465:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/JavaScriptCore/runtime/JSArray.h:151:  The parameter name "exec" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:152:  The parameter name "exec" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:152:  The parameter name "storage" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:156:  The parameter name "callType" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:156:  The parameter name "callData" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:202:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Total errors found: 12 in 52 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 18 Early Warning System Bot 2012-10-06 15:49:16 PDT 
Comment on attachment 167462 [details]
the patch

Attachment 167462 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/14207063
Comment 19 Early Warning System Bot 2012-10-06 15:51:49 PDT 
Comment on attachment 167462 [details]
the patch

Attachment 167462 [details] did not pass qt-wk2-ews (qt):
Output: http://queues.webkit.org/results/14181802
Comment 20 Build Bot 2012-10-06 15:55:39 PDT 
Comment on attachment 167462 [details]
the patch

Attachment 167462 [details] did not pass win-ews (win):
Output: http://queues.webkit.org/results/14181806
Comment 21 Gyuyoung Kim 2012-10-06 16:07:54 PDT 
Comment on attachment 167462 [details]
the patch

Attachment 167462 [details] did not pass efl-ews (efl):
Output: http://queues.webkit.org/results/14183812
Comment 22 Build Bot 2012-10-06 19:01:37 PDT 
Comment on attachment 167462 [details]
the patch

Attachment 167462 [details] did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/14181834

New failing tests:
fast/js/array-bad-time.html
fast/js/cross-frame-really-bad-time-with-__proto__.html
fast/js/mozilla/strict/15.4.4.12.html
fast/js/array-slow-put.html
fast/js/dfg-array-push-bad-time.html
fast/js/cross-frame-really-bad-time.html
fast/js/array-defineOwnProperty.html
sputnik/Conformance/15_Native_Objects/15.4_Array/15.4.4/15.4.4.6_Array_prototype_pop/S15.4.4.6_A4_T1.html
fast/js/Object-defineProperty.html
fast/dom/message-port-deleted-by-accessor.html
fast/canvas/webgl/shader-deleted-by-accessor.html
fast/js/dfg-holy-put-by-val-interferes-with-get-array-length.html
fast/js/dfg-array-push-slow-put.html
fast/js/object-bad-time.html
fast/js/dfg-array-pop-side-effects.html
fast/js/mozilla/strict/15.4.4.6.html
Comment 23 Geoffrey Garen 2012-10-07 16:42:17 PDT 
Comment on attachment 167462 [details]
the patch

Seems to be failing all the things.
Comment 24 Filip Pizlo 2012-10-07 22:03:22 PDT 
Created attachment 167502 [details]
the patch

Still need to make the various build systems work but this one should be fine other than that.
Comment 25 WebKit Review Bot 2012-10-07 22:06:46 PDT 
Attachment 167502 [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/runtime/JSObject.h:695:  The parameter name "butterfly" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSObject.h:764:  The parameter name "globalData" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/dfg/DFGOperations.cpp:609:  When wrapping a line, only indent 4 spaces.  [whitespace/indent] [3]
Source/JavaScriptCore/runtime/JSArray.cpp:1120:  Should have only a single space after a punctuation in a comment.  [whitespace/comments] [5]
Source/JavaScriptCore/jit/JITPropertyAccess.cpp:1430:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/JavaScriptCore/jit/JITPropertyAccess.cpp:1465:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/JavaScriptCore/runtime/JSArray.h:151:  The parameter name "exec" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:152:  The parameter name "exec" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:152:  The parameter name "storage" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:156:  The parameter name "callType" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:156:  The parameter name "callData" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:202:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Total errors found: 12 in 52 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 26 Early Warning System Bot 2012-10-07 22:20:26 PDT 
Comment on attachment 167502 [details]
the patch

Attachment 167502 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/14221169
Comment 27 Early Warning System Bot 2012-10-07 22:23:40 PDT 
Comment on attachment 167502 [details]
the patch

Attachment 167502 [details] did not pass qt-wk2-ews (qt):
Output: http://queues.webkit.org/results/14223141
Comment 28 Build Bot 2012-10-07 22:29:38 PDT 
Comment on attachment 167502 [details]
the patch

Attachment 167502 [details] did not pass win-ews (win):
Output: http://queues.webkit.org/results/14228125
Comment 29 Gyuyoung Kim 2012-10-07 22:30:13 PDT 
Comment on attachment 167502 [details]
the patch

Attachment 167502 [details] did not pass efl-ews (efl):
Output: http://queues.webkit.org/results/14218180
Comment 30 Filip Pizlo 2012-10-08 00:10:43 PDT 
Created attachment 167513 [details]
the patch

It should now pass all of the things, at least on Mac.  We'll see how things go on other build systems.
Comment 31 WebKit Review Bot 2012-10-08 00:13:41 PDT 
Attachment 167513 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/CMakeLists.txt', u'S..." exit_code: 1
Source/JavaScriptCore/runtime/JSObject.h:696:  The parameter name "butterfly" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSObject.h:765:  The parameter name "globalData" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/dfg/DFGOperations.cpp:609:  When wrapping a line, only indent 4 spaces.  [whitespace/indent] [3]
Source/JavaScriptCore/runtime/JSArray.cpp:1120:  Should have only a single space after a punctuation in a comment.  [whitespace/comments] [5]
Source/JavaScriptCore/jit/JITPropertyAccess.cpp:1430:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/JavaScriptCore/jit/JITPropertyAccess.cpp:1465:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/JavaScriptCore/runtime/JSArray.h:157:  The parameter name "exec" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:158:  The parameter name "exec" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:158:  The parameter name "storage" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:162:  The parameter name "callType" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:162:  The parameter name "callData" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:208:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Total errors found: 12 in 57 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 32 Gyuyoung Kim 2012-10-08 00:21:24 PDT 
Comment on attachment 167513 [details]
the patch

Attachment 167513 [details] did not pass efl-ews (efl):
Output: http://queues.webkit.org/results/14199008
Comment 33 Early Warning System Bot 2012-10-08 00:26:27 PDT 
Comment on attachment 167513 [details]
the patch

Attachment 167513 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/14204564
Comment 34 kov's GTK+ EWS bot 2012-10-08 00:26:31 PDT 
Comment on attachment 167513 [details]
the patch

Attachment 167513 [details] did not pass gtk-ews (gtk):
Output: http://queues.webkit.org/results/14105048
Comment 35 Early Warning System Bot 2012-10-08 00:29:53 PDT 
Comment on attachment 167513 [details]
the patch

Attachment 167513 [details] did not pass qt-wk2-ews (qt):
Output: http://queues.webkit.org/results/14205572
Comment 36 Filip Pizlo 2012-10-08 01:03:49 PDT 
Created attachment 167517 [details]
the patch
Comment 37 WebKit Review Bot 2012-10-08 01:07:26 PDT 
Attachment 167517 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/CMakeLists.txt', u'S..." exit_code: 1
Source/JavaScriptCore/runtime/JSObject.h:696:  The parameter name "butterfly" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSObject.h:765:  The parameter name "globalData" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/dfg/DFGOperations.cpp:609:  When wrapping a line, only indent 4 spaces.  [whitespace/indent] [3]
Source/JavaScriptCore/runtime/JSArray.cpp:1120:  Should have only a single space after a punctuation in a comment.  [whitespace/comments] [5]
Source/JavaScriptCore/jit/JITPropertyAccess.cpp:1430:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/JavaScriptCore/jit/JITPropertyAccess.cpp:1465:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/JavaScriptCore/runtime/JSArray.h:157:  The parameter name "exec" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:158:  The parameter name "exec" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:158:  The parameter name "storage" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:162:  The parameter name "callType" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:162:  The parameter name "callData" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:208:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Total errors found: 12 in 57 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 38 Early Warning System Bot 2012-10-08 01:19:13 PDT 
Comment on attachment 167517 [details]
the patch

Attachment 167517 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/14196815
Comment 39 Early Warning System Bot 2012-10-08 01:24:17 PDT 
Comment on attachment 167517 [details]
the patch

Attachment 167517 [details] did not pass qt-wk2-ews (qt):
Output: http://queues.webkit.org/results/14221201
Comment 40 Build Bot 2012-10-08 01:35:19 PDT 
Comment on attachment 167517 [details]
the patch

Attachment 167517 [details] did not pass win-ews (win):
Output: http://queues.webkit.org/results/14209382
Comment 41 Filip Pizlo 2012-10-08 13:45:20 PDT 
Created attachment 167599 [details]
the patch
Comment 42 WebKit Review Bot 2012-10-08 13:48:14 PDT 
Attachment 167599 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/CMakeLists.txt', u'S..." exit_code: 1
Source/JavaScriptCore/runtime/JSObject.h:696:  The parameter name "butterfly" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSObject.h:765:  The parameter name "globalData" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/dfg/DFGOperations.cpp:609:  When wrapping a line, only indent 4 spaces.  [whitespace/indent] [3]
Source/JavaScriptCore/runtime/JSArray.cpp:1120:  Should have only a single space after a punctuation in a comment.  [whitespace/comments] [5]
Source/JavaScriptCore/jit/JITPropertyAccess.cpp:1430:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/JavaScriptCore/jit/JITPropertyAccess.cpp:1465:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/JavaScriptCore/runtime/JSArray.h:157:  The parameter name "exec" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:158:  The parameter name "exec" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:158:  The parameter name "storage" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:162:  The parameter name "callType" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:162:  The parameter name "callData" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:208:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Total errors found: 12 in 57 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 43 Build Bot 2012-10-08 14:29:34 PDT 
Comment on attachment 167599 [details]
the patch

Attachment 167599 [details] did not pass win-ews (win):
Output: http://queues.webkit.org/results/14207627
Comment 44 Filip Pizlo 2012-10-08 16:51:28 PDT 
Created attachment 167647 [details]
the patch
Comment 45 WebKit Review Bot 2012-10-08 16:58:27 PDT 
Attachment 167647 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/CMakeLists.txt', u'S..." exit_code: 1
Source/JavaScriptCore/runtime/JSObject.h:696:  The parameter name "butterfly" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSObject.h:765:  The parameter name "globalData" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/dfg/DFGOperations.cpp:609:  When wrapping a line, only indent 4 spaces.  [whitespace/indent] [3]
Source/JavaScriptCore/runtime/JSArray.cpp:1120:  Should have only a single space after a punctuation in a comment.  [whitespace/comments] [5]
Source/JavaScriptCore/jit/JITPropertyAccess.cpp:1430:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/JavaScriptCore/jit/JITPropertyAccess.cpp:1465:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Source/JavaScriptCore/runtime/JSArray.h:157:  The parameter name "exec" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:158:  The parameter name "exec" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:158:  The parameter name "storage" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:162:  The parameter name "callType" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:162:  The parameter name "callData" adds no information, so it should be removed.  [readability/parameter_name] [5]
Source/JavaScriptCore/runtime/JSArray.h:208:  Weird number of spaces at line-start.  Are you using a 4-space indent?  [whitespace/indent] [3]
Total errors found: 12 in 57 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 46 Build Bot 2012-10-08 19:28:50 PDT 
Comment on attachment 167647 [details]
the patch

Attachment 167647 [details] did not pass win-ews (win):
Output: http://queues.webkit.org/results/14211595
Comment 47 Mark Hahnenberg 2012-10-09 13:43:52 PDT 
Comment on attachment 167647 [details]
the patch

r=me with style and build fixes
Comment 48 Filip Pizlo 2012-10-09 14:08:28 PDT 
Created attachment 167844 [details]
patch for landing

Fixing things.
Comment 49 WebKit Review Bot 2012-10-09 14:11:29 PDT 
Attachment 167844 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/CMakeLists.txt', u'S..." exit_code: 1
Source/JavaScriptCore/dfg/DFGOperations.cpp:609:  When wrapping a line, only indent 4 spaces.  [whitespace/indent] [3]
Total errors found: 1 in 58 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 50 Filip Pizlo 2012-10-09 16:41:08 PDT 
Landed in http://trac.webkit.org/changeset/130826
Comment 51 Csaba Osztrogonác 2012-10-09 22:16:49 PDT 
(In reply to comment #50)
> Landed in http://trac.webkit.org/changeset/130826
... and buildfix landed in http://trac.webkit.org/changeset/130826
... and it caused a serious regression on Qt-ARM: https://bugs.webkit.org/show_bug.cgi?id=98857
Comment 52 Csaba Osztrogonác 2012-10-09 22:37:07 PDT 
One more regression on Qt-WK2 64 bit: https://bugs.webkit.org/show_bug.cgi?id=98859