- 31 Jan, 2013 2 commits
-
-
danno@chromium.org authored
Port r13557 (aa3ba8a) BUG= Review URL: https://codereview.chromium.org/12091083 Patch from Paul Lind <plind44@gmail.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13567 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
verwaest@chromium.org authored
transitioned from a map with a different descriptor array (or has no back pointer), mark all valid descriptors from the start. This fixes the following memory leak: Map A shares a descriptor array with map B. Map B adds constant function c that in its scope holds on to an instance of B. If the descriptor array of A would keep all the shared descriptors alive, including c, this keeps alive both A and c indefinitely. This CL also fixes a bug in descriptor array trimming. When trimming descriptor arrays we need to trim off the slack as well (thus the entire storage); and since we are trimming a descriptor array, we need to trim * kDescriptorSize. Review URL: https://chromiumcodereview.appspot.com/12084066 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13566 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 30 Jan, 2013 16 commits
-
-
adamk@chromium.org authored
When a data property has its attributes changed but its value remains the same, don't emit an oldValue. This makes the API more consistent by only emitting oldValue when the value of a property has actually changed (or been removed, in the case of a reconfiguration as an accessor property or a deletion). Review URL: https://codereview.chromium.org/11820004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13565 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
jkummerow@chromium.org authored
Review URL: https://codereview.chromium.org/12090072 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13562 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
Port r13542 (d3217e18) BUG= TEST= Review URL: https://codereview.chromium.org/12091042 Patch from Akos Palfi <palfia@homejinni.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13561 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
Port r13533 (2f339757) Original commit message: In preparation of supporting stubs that deopt and then need to push their register-based parameters to an arguments area on the stack that gets properly collected, add StubFailureTrampolineFrames to hold those parameters. BUG= TEST= Review URL: https://codereview.chromium.org/12087053 Patch from Akos Palfi <palfia@homejinni.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13560 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
hpayer@chromium.org authored
BUG= Review URL: https://codereview.chromium.org/12079076 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13559 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
verwaest@chromium.org authored
With the old implementation, due to the map-check being inadequate, such transitions were already added for cross-context field stores. It is not necessary anymore to not store transitions, since we properly clear non-live transitions. Globally enabling this feature will help find more bugs. BUG=v8:2518 R=mstarzinger@chromium.org Review URL: https://chromiumcodereview.appspot.com/12092063 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13558 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
R=yangguo@chromium.org Review URL: https://codereview.chromium.org/12084063 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13557 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
hpayer@chromium.org authored
BUG= Review URL: https://codereview.chromium.org/12079075 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13554 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
hpayer@chromium.org authored
BUG= Review URL: https://codereview.chromium.org/12094057 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13553 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
hpayer@chromium.org authored
Sweep old pointer space and old data space concurrently to the main mutator thread and in parallel. BUG= Review URL: https://codereview.chromium.org/11782028 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13552 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
hpayer@chromium.org authored
Precisely measure duration of mark and sweep phases. Changed print_cumulative_gc_stat flag to only print GC statistics at end of program. BUG= Review URL: https://codereview.chromium.org/11595006 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13551 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ulan@chromium.org authored
This lets the Deoptimizer pointer stay in r0, saving a mov and simplifying the code a little. BUG=none TEST=Tested with test/mjsunit/math-floor-of-div.js, this fails reliably if OSR is broken Review URL: https://chromiumcodereview.appspot.com/12082046 Patch from Hans Wennborg <hans@chromium.org>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13550 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=verwaest@chromium.org Review URL: https://codereview.chromium.org/12094038 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13549 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ulan@chromium.org authored
This also fixes a bug caused by a missing comma after "d13" in the 'names' array in AllocationIndexToString. BUG=none Review URL: https://chromiumcodereview.appspot.com/12082044 Patch from Hans Wennborg <hans@chromium.org>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13548 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ulan@chromium.org authored
This matches Simulator::registers_ and is in line with the style guide. BUG=none Review URL: https://chromiumcodereview.appspot.com/12096036 Patch from Hans Wennborg <hans@chromium.org>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13547 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/12087081 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13544 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 29 Jan, 2013 12 commits
-
-
mmassi@chromium.org authored
Review URL: https://codereview.chromium.org/12090021 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13543 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/12096040 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13542 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
R=jkummerow@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/12077043 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13541 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
Make --prof_lazy log the addresses of shared libraries. Without those addresses, the output of the tick processor is not very useful for node.js applications. Address logging was disabled for --prof-lazy in r3154 for reasons that no longer apply: the profiler is started on demand and hence looking up the addresses is deferred until it's actually necessary. Review URL: https://codereview.chromium.org/11309007 Patch from Ben Noordhuis <info@bnoordhuis.nl>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13540 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/12094032 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13539 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
TBR=danno@chromium.org Review URL: https://codereview.chromium.org/12093041 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13538 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
verwaest@chromium.org authored
Review URL: https://chromiumcodereview.appspot.com/12079024 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13537 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
jkummerow@chromium.org authored
Review URL: https://codereview.chromium.org/12095012 Patch from Haitao Feng <haitao.feng@intel.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13536 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
TBR=danno@chromium.org Review URL: https://codereview.chromium.org/12095033 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13535 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
Improved register usage a bit on the way. Review URL: https://codereview.chromium.org/12082043 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13534 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
In preparation of supporting stubs that deopt and then need to push their register-based parameters to an arguments area on the stack that gets properly collected, add StubFailureTrampolineFrames to hold those parameters. R=mstarzinger@chromium.org Review URL: https://chromiumcodereview.appspot.com/12052053 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13533 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
This is done by first committing the deoptimization entry code with a minimal area size (OS::CommitPageSize) and later using CommitArea to adjust the size. Committed: http://code.google.com/p/v8/source/detail?r=13494 Review URL: https://chromiumcodereview.appspot.com/11566011 Patch from Haitao Feng <haitao.feng@intel.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13532 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 28 Jan, 2013 7 commits
-
-
svenpanne@chromium.org authored
Review URL: https://codereview.chromium.org/12093016 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13531 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
verwaest@chromium.org authored
Review URL: https://chromiumcodereview.appspot.com/12088021 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13530 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
R=haraken@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/12040075 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13529 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
verwaest@chromium.org authored
Port r13506 (0d666576) BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/12077009 Patch from Akos Palfi <palfia@homejinni.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13528 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
The previous duplication is quite bad from an architectural point of view. Furthermore, it messes up the output of --hydrogen-stats. As remarked in a comment, there is still more unification work to do, but at least this CL is a step in the right direction... Review URL: https://codereview.chromium.org/12091016 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13527 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
verwaest@chromium.org authored
This is necessary for monomorphic stubs that rely on instance types, such as ArrayLength, StringLength and FunctionPrototype. BUG=chromium:172345 Review URL: https://chromiumcodereview.appspot.com/12082023 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13526 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ulan@chromium.org authored
Disable the flag by default because of Chrome crashes. BUG=172488,172489 R=mstarzinger@chromium.org Review URL: https://chromiumcodereview.appspot.com/12091018 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13523 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 25 Jan, 2013 3 commits
-
-
mstarzinger@chromium.org authored
This allows Crankshaft to allow inlining of multiple different closures that were all derived from the same shared function info. This pattern appears when libraries provide generic closures that are used over and over again at different call-sites. R=jkummerow@chromium.org TEST=mjsunit/compiler/inline-closures Review URL: https://codereview.chromium.org/12071002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13522 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
R=jkummerow@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/12039088 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13521 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
Replaced a scary pointer comparison by strcmp on the way. Review URL: https://codereview.chromium.org/12035101 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13510 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-