Summary: | JSC should infer when indexed storage contains only integers or doubles | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Filip Pizlo <fpizlo> | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Component: | JavaScriptCore | Assignee: | Filip Pizlo <fpizlo> | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Severity: | Normal | CC: | barraclough, dino, 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: | 99263, 99371, 97288, 98997, 99260, 99261, 99262, 99269, 99287, 99557, 99905, 100052, 100311, 100328, 100599, 100620, 100827, 101174, 101704, 101706, 101740, 101746, 101832 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Bug Blocks: | 100870, 101145, 101718 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Attachments: |
|
Description
Filip Pizlo
2012-10-06 15:55:26 PDT
Created attachment 169066 [details]
work in progress
Created attachment 169069 [details]
moar
Created attachment 169493 [details]
more things
I think I have a story for array allocation inference.
Created attachment 169678 [details]
even more things
Created attachment 169757 [details]
more profiling
I think that the array allocation profiling story is now done. All LLInt and baseline JIT array allocation sites should be doing the right thing, and the GC should be helping out.
(I'm speaking as if the code is doing things. It's not. It doesn't compile yet.)
Created attachment 169771 [details]
done with 64-bit LLInt and baseline JIT
Now I just need to write the DFG part! And the 32-bit parts. And get the other platforms building.
Created attachment 170026 [details]
did more things
Started doing the DFG stuff.
Created attachment 170475 [details]
bigger is better!
Created attachment 170545 [details]
more!
Lots of meaty DFG stuff done.
Created attachment 170699 [details]
I think the 64-bit bit code is complete
Still haven't tried compiling it.
Created attachment 170842 [details]
it compiles
It compiles.
Created attachment 171066 [details]
it runs things
Looks like 54% speed-up on doubly programs and a 14% speed-up on inty programs.
Created attachment 171111 [details]
it speeds up real programs
Still working on some regressions, though.
Created attachment 171125 [details]
rebased and fighting regressions
There is still some weirdness in the profiling. Figuring it out now.
Created attachment 171351 [details]
it seems to be passing some tests
So far: 1% regression on SunSpider, >2% progression on V8v7, >2% progression on Kraken.
Created attachment 171507 [details]
more tests pass
Created attachment 171547 [details]
almost done with 32-bit
Still a bit more work to do though.
Created attachment 171759 [details]
it appears to totally work
Will mark r? as soon as I finish a few more tests.
Attachment 171759 [details] did not pass style-queue:
Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/API/JSObjectRef.cpp'..." exit_code: 1
Source/JavaScriptCore/runtime/JSObject.h:956: The parameter name "globalData" adds no information, so it should be removed. [readability/parameter_name] [5]
Source/JavaScriptCore/bytecode/Instruction.h:50: Code inside a namespace should not be indented. [whitespace/indent] [4]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1852: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1872: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1876: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1881: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1888: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1890: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1896: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1899: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1906: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2067: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2071: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2080: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2082: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2089: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2092: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2095: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2100: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:48: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:49: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:56: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:57: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:58: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Total errors found: 24 in 81 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 171759 [details] it appears to totally work Attachment 171759 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/14693349 Comment on attachment 171759 [details] it appears to totally work Attachment 171759 [details] did not pass qt-wk2-ews (qt): Output: http://queues.webkit.org/results/14697131 Comment on attachment 171759 [details] it appears to totally work Attachment 171759 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/14663402 Comment on attachment 171759 [details] it appears to totally work Attachment 171759 [details] did not pass efl-ews (efl): Output: http://queues.webkit.org/results/14678366 Comment on attachment 171759 [details]
it appears to totally work
This is now passing 32-bit and 64-bit tests for me. It appears there are some style errors (I will fix the real ones) and I still need to update build files for Qt, Gtk, Win, and EFL. But, the patch is in good shape other than that and so it's ready for review!
Created attachment 171934 [details]
build systems should be in order
Fixed build systems. Random 32-bit fixes.
Created attachment 171941 [details]
rebase
Attachment 171941 [details] did not pass style-queue:
Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/API/JSObjectRef.cpp'..." exit_code: 1
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2067: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2071: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2080: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2082: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2089: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2092: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2095: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2100: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/runtime/JSObject.h:956: The parameter name "globalData" adds no information, so it should be removed. [readability/parameter_name] [5]
Source/JavaScriptCore/bytecode/Instruction.h:50: Code inside a namespace should not be indented. [whitespace/indent] [4]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1852: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1872: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1876: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1881: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1888: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1890: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1896: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1899: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1906: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:48: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:49: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:56: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:57: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:58: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Total errors found: 24 in 84 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 171941 [details] rebase Attachment 171941 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/14678663 Comment on attachment 171941 [details] rebase View in context: https://bugs.webkit.org/attachment.cgi?id=171941&action=review Thanks for taking care! > Source/JavaScriptCore/dfg/DFGSpeculativeJIT64.cpp:3689 > + m_jit.move(TrustedImm64(bitwise_cast<intptr_t>(0.0 / 0.0)), scratchGPR); Can we use bitwise_cast<int64_t>? x32 would be happy for this. :) > Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:85 > + m_jit.move(TrustedImm64(bitwise_cast<intptr_t>(0.0 / 0.0)), scratchGPR); Can we use bitwise_cast<int64_t>? x32 would be happy for this. :) (In reply to comment #29) > (From update of attachment 171941 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=171941&action=review > > Thanks for taking care! > > > Source/JavaScriptCore/dfg/DFGSpeculativeJIT64.cpp:3689 > > + m_jit.move(TrustedImm64(bitwise_cast<intptr_t>(0.0 / 0.0)), scratchGPR); > > Can we use bitwise_cast<int64_t>? x32 would be happy for this. :) > > > Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:85 > > + m_jit.move(TrustedImm64(bitwise_cast<intptr_t>(0.0 / 0.0)), scratchGPR); > > Can we use bitwise_cast<int64_t>? x32 would be happy for this. :) Yup, I can make these changes. Created attachment 172173 [details]
fixes and rebasing
Attachment 172173 [details] did not pass style-queue:
Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/API/JSObjectRef.cpp'..." exit_code: 1
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2067: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2071: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2080: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2082: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2089: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2092: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2095: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2100: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/runtime/JSObject.h:956: The parameter name "globalData" adds no information, so it should be removed. [readability/parameter_name] [5]
Source/JavaScriptCore/bytecode/Instruction.h:50: Code inside a namespace should not be indented. [whitespace/indent] [4]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1852: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1872: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1876: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1881: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1888: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1890: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1896: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1899: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1906: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:48: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:49: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:56: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:57: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:58: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Total errors found: 24 in 84 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 172173 [details] fixes and rebasing Attachment 172173 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/14719072 Created attachment 172249 [details]
bigger speed-up
Now up to 4.8% faster on V8v7.
Attachment 172249 [details] did not pass style-queue:
Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/API/JSObjectRef.cpp'..." exit_code: 1
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2067: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2071: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2080: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2082: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2089: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2092: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2095: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2100: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/runtime/JSObject.h:956: The parameter name "globalData" adds no information, so it should be removed. [readability/parameter_name] [5]
Source/JavaScriptCore/bytecode/Instruction.h:50: Code inside a namespace should not be indented. [whitespace/indent] [4]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1852: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1872: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1876: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1881: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1888: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1890: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1896: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1899: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1906: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:48: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:49: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:56: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:57: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:58: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Total errors found: 24 in 84 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 172249 [details] bigger speed-up Attachment 172249 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/14744016 (In reply to comment #36) > (From update of attachment 172249 [details]) > Attachment 172249 [details] did not pass win-ews (win): > Output: http://queues.webkit.org/results/14744016 Looks like VC++ is too pedantic to compile 0.0/0.0. Weird. I will fix by switching our references to NaN to a #define. See https://bugs.webkit.org/show_bug.cgi?id=101174 Created attachment 172362 [details]
rebase and address Windows fail
Attachment 172362 [details] did not pass style-queue:
Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/API/JSObjectRef.cpp'..." exit_code: 1
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2067: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2071: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2080: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2082: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2089: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2092: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2095: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2100: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/runtime/JSObject.h:956: The parameter name "globalData" adds no information, so it should be removed. [readability/parameter_name] [5]
Source/JavaScriptCore/bytecode/Instruction.h:50: Code inside a namespace should not be indented. [whitespace/indent] [4]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1852: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1872: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1876: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1881: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1888: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1890: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1896: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1899: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1906: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:48: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:49: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:56: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:57: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:58: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Total errors found: 24 in 84 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 172362 [details] rebase and address Windows fail Attachment 172362 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/14744204 Created attachment 172458 [details]
more Windows fixes
Attachment 172458 [details] did not pass style-queue:
Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/API/JSObjectRef.cpp'..." exit_code: 1
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2067: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2071: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2080: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2082: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2089: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2092: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2095: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2100: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/runtime/JSObject.h:955: The parameter name "globalData" adds no information, so it should be removed. [readability/parameter_name] [5]
Source/JavaScriptCore/bytecode/Instruction.h:50: Code inside a namespace should not be indented. [whitespace/indent] [4]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1852: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1872: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1876: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1881: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1888: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1890: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1896: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1899: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1906: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:48: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:49: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:56: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:57: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:58: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Total errors found: 24 in 85 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 172479 [details]
rebase
Created attachment 172493 [details]
rebase
Attachment 172493 [details] did not pass style-queue:
Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/API/JSObjectRef.cpp'..." exit_code: 1
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2067: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2071: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2080: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2082: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2089: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2092: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2095: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2100: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/Instruction.h:50: Code inside a namespace should not be indented. [whitespace/indent] [4]
Source/JavaScriptCore/bytecode/ArrayProfile.h:48: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:49: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:56: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:57: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:58: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/runtime/JSObject.h:955: The parameter name "globalData" adds no information, so it should be removed. [readability/parameter_name] [5]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1852: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1872: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1876: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1881: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1888: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1890: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1896: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1899: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1906: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Total errors found: 24 in 86 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 172625 [details]
rebase
Attachment 172625 [details] did not pass style-queue:
Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/API/JSObjectRef.cpp'..." exit_code: 1
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2067: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2071: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2080: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2082: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2089: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2092: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2095: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2100: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/Instruction.h:50: Code inside a namespace should not be indented. [whitespace/indent] [4]
Source/JavaScriptCore/bytecode/ArrayProfile.h:48: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:49: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:56: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:57: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:58: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/runtime/JSObject.h:955: The parameter name "globalData" adds no information, so it should be removed. [readability/parameter_name] [5]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1852: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1872: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1876: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1881: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1888: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1890: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1896: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1899: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1906: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Total errors found: 24 in 86 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 172625 [details] rebase Attachment 172625 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/14744479 Comment on attachment 172625 [details] rebase View in context: https://bugs.webkit.org/attachment.cgi?id=172625&action=review r=me, one concern i have (although not introduced by this patch) is that there are a number of places where we are multiplying an attacker controlled value and using that to control an allocation size. I don't think op_new_array_buffer really needs a profile as we should be able to just record that when we originally generate the instruction. Also couldn't we use this information to improve the GC? If we know the indexing type is int or double then there is no need to mark the array storage. Anyhoo, r=me with the exception of windows bustage this seems fine to land. > Source/JavaScriptCore/jit/JITExceptions.cpp:45 > ExceptionHandler genericThrow(JSGlobalData* globalData, ExecState* callFrame, JSValue exceptionValue, unsigned vPCIndex) > { > ASSERT(exceptionValue); > - > + > globalData->exception = JSValue(); > HandlerInfo* handler = globalData->interpreter->throwException(callFrame, exceptionValue, vPCIndex); // This may update callFrame & exceptionValue! > globalData->exception = exceptionValue; Reverteration here (In reply to comment #49) > (From update of attachment 172625 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=172625&action=review > > r=me, one concern i have (although not introduced by this patch) is that there are a number of places where we are multiplying an attacker controlled value and using that to control an allocation size. > > I don't think op_new_array_buffer really needs a profile as we should be able to just record that when we originally generate the instruction. It does need the profile. Consider: Some thingy: var a = [1,2,3]; Some other thingy: a[4] = 0.5; Array allocation profiling will hint to us that we should allocate 'a' as an array of doubles, if the other thingy happens often enough. > > Also couldn't we use this information to improve the GC? If we know the indexing type is int or double then there is no need to mark the array storage. This patch already does that! :-) > > Anyhoo, r=me with the exception of windows bustage this seems fine to land. > > > Source/JavaScriptCore/jit/JITExceptions.cpp:45 > > ExceptionHandler genericThrow(JSGlobalData* globalData, ExecState* callFrame, JSValue exceptionValue, unsigned vPCIndex) > > { > > ASSERT(exceptionValue); > > - > > + > > globalData->exception = JSValue(); > > HandlerInfo* handler = globalData->interpreter->throwException(callFrame, exceptionValue, vPCIndex); // This may update callFrame & exceptionValue! > > globalData->exception = exceptionValue; > > Reverteration here Ackitude. Created attachment 172680 [details]
patch for landing
Fixing windows
Created attachment 172686 [details]
patch for landing
Rebased past Oliver's monster change.
Attachment 172686 [details] did not pass style-queue:
Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/API/JSObjectRef.cpp'..." exit_code: 1
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2067: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2071: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2080: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2082: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2089: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2092: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2095: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2100: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/Instruction.h:50: Code inside a namespace should not be indented. [whitespace/indent] [4]
Source/JavaScriptCore/bytecode/ArrayProfile.h:48: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:49: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:56: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:57: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:58: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/runtime/JSObject.h:955: The parameter name "globalData" adds no information, so it should be removed. [readability/parameter_name] [5]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1852: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1872: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1876: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1881: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1888: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1890: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1896: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1899: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1906: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Total errors found: 24 in 88 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 172686 [details] patch for landing Attachment 172686 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/14744593 Created attachment 172731 [details]
fix more windows
Attachment 172731 [details] did not pass style-queue:
Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'Source/JavaScriptCore/API/JSObjectRef.cpp'..." exit_code: 1
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2067: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2071: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2080: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2082: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2089: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2092: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2095: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT32_64.cpp:2100: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/Instruction.h:50: Code inside a namespace should not be indented. [whitespace/indent] [4]
Source/JavaScriptCore/bytecode/ArrayProfile.h:48: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:49: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:56: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:57: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/bytecode/ArrayProfile.h:58: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3]
Source/JavaScriptCore/runtime/JSObject.h:955: The parameter name "globalData" adds no information, so it should be removed. [readability/parameter_name] [5]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1852: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1872: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1876: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1881: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1888: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1890: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1896: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1899: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Source/JavaScriptCore/dfg/DFGSpeculativeJIT.cpp:1906: When wrapping a line, only indent 4 spaces. [whitespace/indent] [3]
Total errors found: 24 in 89 files
If any of these errors are false positives, please file a bug against check-webkit-style.
Landed in http://trac.webkit.org/changeset/133953 (In reply to comment #57) > Landed in http://trac.webkit.org/changeset/133953 It broke the MIPS build: https://bugs.webkit.org/show_bug.cgi?id=101704 and the ARM_TRADITIONAL build: https://bugs.webkit.org/show_bug.cgi?id=101706 (In reply to comment #57) > Landed in http://trac.webkit.org/changeset/133953 It broke the build on ARM and MIPS - https://bugs.webkit.org/show_bug.cgi?id=101740 Could you review the proposed patches? |