- 21 Oct, 2013 19 commits
-
-
palfia@homejinni.com authored
Port r17295 (b36eb655) Original commit message: In the process: - Add a command-line flag --opt-code-positions to track source position information throughout optimized code. - Add a subclass of the hydrogen graph builder to ensure that the source position is properly set on the graph builder for all generated hydrogen code. - Overhaul handling of source positions in hydrogen to ensure they are passed through to generated code consistently and in most cases transparently. BUG= R=plind44@gmail.com Review URL: https://codereview.chromium.org/33803002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17305 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
palfia@homejinni.com authored
Port r17290 (b5fabb92) Original commit message: This fixes a performance regression that was caused by converting the BinaryOpStub to a Hydrogen code stub. It also fixes a leftover TODO wrt. the handling of Number*String or String*Number versions of the stub. BUG= R=plind44@gmail.com Review URL: https://codereview.chromium.org/32473007 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17304 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yurys@chromium.org authored
The test started failing on V8 Linux - nosnap - debug after r17301 which added stack traversal to the allocation tracker. I'm disabling it for now until the problem is fixed. BUG=v8:2949 TBR=hpayer@chromium.org Review URL: https://codereview.chromium.org/30633005 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17303 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yurys@chromium.org authored
This is initial implementation of allocation profiler. Whenever new object allocation is reported to the HeapProfiler and allocation tracking is on we will capture current stack trace, add it to the collection of the allocation traces (a tree) and attribute the allocated size to the top JS function on the stack. Format of serialized heap snapshot is extended to include information about recorded allocation stack traces. BUG=chromium:277984 R=hpayer@chromium.org, loislo@chromium.org Review URL: https://codereview.chromium.org/27227005 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17301 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ishell@chromium.org authored
R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/32513002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17300 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
titzer@chromium.org authored
BUG= R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/31943002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17299 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
This finally gets rid of the pesky trampoline in SetValueInferType and enforces the layering between PropertyCell and Heap. It requires full handlification of NewGlobalObject as well, which is only used when the snapshot is created at compile-time. R=rossberg@chromium.org Review URL: https://codereview.chromium.org/28783002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17297 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
bmeurer@chromium.org authored
TBR=danno@chromium.org Review URL: https://codereview.chromium.org/32523002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17296 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
In the process: - Add a command-line flag --opt-code-positions to track source position information throughout optimized code. - Add a subclass of the hydrogen graph builder to ensure that the source position is properly set on the graph builder for all generated hydrogen code. - Overhaul handling of source positions in hydrogen to ensure they are passed through to generated code consistently and in most cases transparently. Originally reviewed in this CL: https://codereview.chromium.org/24957003/ Review URL: https://codereview.chromium.org/29123008 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17295 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
titzer@chromium.org authored
Handle misaligned loads and stores in load elimination. Do not track misaligned loads and be conservative about invalidating misaligned stores. Add more tests for number conversion to string (NumberToStringStub exhibits misaligned loads) BUG=v8:2934 R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/28383003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17294 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
bmeurer@chromium.org authored
R=mstarzinger@chromium.org Review URL: https://codereview.chromium.org/32243003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17291 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
bmeurer@chromium.org authored
This fixes a performance regression that was caused by converting the BinaryOpStub to a Hydrogen code stub. It also fixes a leftover TODO wrt. the handling of Number*String or String*Number versions of the stub. R=rossberg@chromium.org Review URL: https://codereview.chromium.org/27674002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17290 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
bmeurer@chromium.org authored
Revert "Add a soft-deopt in keyed element access when current IC is pre-monomorphic and no type feedback was collected." This reverts commit r17288 for breaking the mjsunit/unbox-double-arrays test on almost every platform. TBR=hpayer@chromium.org Review URL: https://codereview.chromium.org/32373002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17289 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
hpayer@chromium.org authored
Add a soft-deopt in keyed element access when current IC is pre-monomorphic and no type feedback was collected. BUG= R=danno@chromium.org Review URL: https://codereview.chromium.org/32263002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17288 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
BUG=v8:2938 R=dslomov@chromium.org Review URL: https://codereview.chromium.org/28793002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17286 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
titzer@chromium.org authored
This reverts commit r17281. TBR=bmeurer@chromium.org Review URL: https://codereview.chromium.org/32113002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17284 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
titzer@chromium.org authored
BUG= R=mstarzinger@chromium.org Review URL: https://codereview.chromium.org/31923003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17281 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
R=dslomov@chromium.org BUG=v8:2938 Review URL: https://codereview.chromium.org/28723002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17279 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
bmeurer@chromium.org authored
R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/31913002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17276 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 18 Oct, 2013 13 commits
-
-
mstarzinger@chromium.org authored
R=rossberg@chromium.org, danno@chromium.org Review URL: https://codereview.chromium.org/28993003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17275 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
titzer@chromium.org authored
BUG= R=mvstanton@chromium.org Review URL: https://codereview.chromium.org/27148004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17273 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
R=ulan@chromium.org BUG= Review URL: https://codereview.chromium.org/27518002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17272 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
loislo@chromium.org authored
In some cases we would like to check untracked objects right from the js code. Otherwise the objects might be collected rigth before check. BUG=none R=hpayer@chromium.org, yurys@chromium.org Review URL: https://codereview.chromium.org/27717003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17270 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
loislo@chromium.org authored
When we have an untracked allocations we need to fix it. It is a bit tricky process due to lack of logging. I added FLAG_heap_profiler_trace_objects that enables log for object movements. Drive by fix: with help of logging I found a bug in LeftTrim method for FixedArray. BUG=none R=svenpanne@chromium.org, yurys@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=17258 Committed: https://code.google.com/p/v8/source/detail?r=17263 Review URL: https://codereview.chromium.org/27694004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17269 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
jkummerow@chromium.org authored
ia32 and x64 more consistent now 1. use non-transcendental cache version of log for x64 2. use negative infinity constant instead of pushing to stack and loading to XMM register 3. remove movdbl, use movsd directly. movdbl seems confusing BUG= R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/27197013 Patch from Weiliang Lin <weiliang.lin2@gmail.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17268 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/28883002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17265 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
This reverts r17263 because of compile failures accross all builds. TBR=loislo@chromium.org Review URL: https://codereview.chromium.org/28773002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17264 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
loislo@chromium.org authored
When we have an untracked allocations we need to fix it. It is a bit tricky process due to lack of logging. I added FLAG_heap_profiler_trace_objects that enables log for object movements. Drive by fix: with help of logging I found a bug in LeftTrim method for FixedArray. BUG=none R=svenpanne@chromium.org, yurys@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=17258 Review URL: https://codereview.chromium.org/27694004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17263 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
R=mstarzinger@chromium.org Review URL: https://codereview.chromium.org/28683002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17262 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yurys@chromium.org authored
Otherwise the string passed as const char* may be disposed and we will end up with a dangling pointer. Also changed StringsStorage::GetCopy so that a copy is not created if the string is already in the cache. BUG=None R=alph@chromium.org, svenpanne@chromium.org Review URL: https://codereview.chromium.org/27627006 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17260 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
loislo@chromium.org authored
Compilation failed on Win64. This reverts commit 6909faccca68c04b7ec4537538e6e7a0cc1650b8. BUG=none TBR=yurys Review URL: https://codereview.chromium.org/28573002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17259 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
loislo@chromium.org authored
When we have an untracked allocations we need to fix it. It is a bit tricky process due to lack of logging. I added FLAG_heap_profiler_trace_objects that enables log for object movements. Drive by fix: with help of logging I found a bug in LeftTrim method for FixedArray. BUG=none R=svenpanne@chromium.org, yurys@chromium.org Review URL: https://codereview.chromium.org/27694004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17258 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 17 Oct, 2013 6 commits
-
-
rafaelw@chromium.org authored
This addresses the leak that mstarzinger points out (https://codereview.chromium.org/26390003/) and includes the test. Note that this adds a test that observing changes to the empty-string property remains possible. BUG= R=mstarzinger@chromium.org, rossberg@chromium.org Review URL: https://codereview.chromium.org/26592012 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17257 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
Use this for mjsunit/unicode-case-overoptimization, which is not related to Crankshaft at all and takes ages. R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/27704002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17255 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
This is a regression test for a bug with handling of count operations that target a JavaScript accessor on the prototype chain in Crankshaft. R=jkummerow@chromium.org BUG=chromium:306851 TEST=mjsunit/regress/regress-crbug-306851 Review URL: https://codereview.chromium.org/27702002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17254 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
marja@chromium.org authored
This is needed for https://codereview.chromium.org/26792002/ BUG= R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/27512003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17253 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
Nobody should need to use $NaN, 0/0, 1/0 and void 0. R=mvstanton@chromium.org BUG= Review URL: https://codereview.chromium.org/27491002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17252 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/27676002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17249 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 16 Oct, 2013 2 commits
-
-
palfia@homejinni.com authored
Port r17239 (c203175f) Original commit message: This is required to be able to use these tokens with HCompareNumericAndBranch. BUG= R=plind44@gmail.com Review URL: https://codereview.chromium.org/27516004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17248 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
R=mvstanton@chromium.org BUG= Review URL: https://codereview.chromium.org/27473006 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17247 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-