Reapply descriptor array sharing. (issue 11093026)

6 views
Skip to first unread message

verw...@chromium.org

unread,
Oct 9, 2012, 12:25:20 PM10/9/12
to jkum...@chromium.org, v8-...@googlegroups.com
Reviewers: Jakob,

Description:
Reapply descriptor array sharing.

This reverts commit 12669


Please review this at https://chromiumcodereview.appspot.com/11093026/

SVN Base: https://v8.googlecode.com/svn/trunk

Affected files:
M src/arm/full-codegen-arm.cc
M src/arm/macro-assembler-arm.h
M src/arm/macro-assembler-arm.cc
M src/bootstrapper.cc
M src/handles.h
M src/handles.cc
M src/heap.h
M src/heap.cc
M src/ia32/full-codegen-ia32.cc
M src/ia32/macro-assembler-ia32.h
M src/ia32/macro-assembler-ia32.cc
M src/mark-compact.cc
M src/objects-debug.cc
M src/objects-inl.h
M src/objects.h
M src/objects.cc
M src/profile-generator.cc
M src/property.h
M src/runtime.cc
M src/string-stream.cc
M src/transitions-inl.h
M src/transitions.h
M src/transitions.cc
M src/version.cc
M src/x64/full-codegen-x64.cc
M src/x64/macro-assembler-x64.h
M src/x64/macro-assembler-x64.cc
M test/cctest/test-heap-profiler.cc
A + test/mjsunit/regress/regress-cntl-descriptors-enum.js
A + test/mjsunit/regress/regress-crbug-148376.js


jkum...@chromium.org

unread,
Oct 9, 2012, 12:54:56 PM10/9/12
to verw...@chromium.org, v8-...@googlegroups.com
LGTM


https://chromiumcodereview.appspot.com/11093026/diff/4001/test/mjsunit/regress/regress-convert-transition.js
File test/mjsunit/regress/regress-convert-transition.js (right):

https://chromiumcodereview.appspot.com/11093026/diff/4001/test/mjsunit/regress/regress-convert-transition.js#newcode28
test/mjsunit/regress/regress-convert-transition.js:28: var input = '{
"a1":1, "a2":1, "a3":1, "a4":1, "a5":1, "a6":1, "a7":1, "a8":1, "a9":1,
"a10":1, "a11":1, "a12":1, "a13":1}';
nit: long line

https://chromiumcodereview.appspot.com/11093026/

verw...@chromium.org

unread,
Oct 9, 2012, 12:59:51 PM10/9/12
to jkum...@chromium.org, v8-...@googlegroups.com
Reply all
Reply to author
Forward
0 new messages