Reviewers: Mads Ager, Erik Corry, Message: The change in test/cctest/test-alloc.cc is due to an unknown issue on the ARM simulator. I will open a bug on this.
Description: Remove the different length string types The different length string types was used to encode the string length and the hash in one field. This is now split into two fields one for length and one for hash. The hash field still encodes the array index of the string if it has one. If an array index is encoded in the hash field the string length is added to the top bits of the hash field to avoid a hash value of zero. On 32-bit this causes an additional 4 bytes to be used for all string objects. On 64-bit this will be half on average dur to pointer alignment. Please review this at http://codereview.chromium.org/436001 SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/ Affected files: M include/v8.h M src/arm/codegen-arm.cc M src/arm/ic-arm.cc M src/arm/stub-cache-arm.cc M src/heap-inl.h M src/heap.h M src/heap.cc M src/ia32/codegen-ia32.cc M src/ia32/ic-ia32.cc M src/ia32/stub-cache-ia32.cc M src/objects-debug.cc M src/objects-inl.h M src/objects.h M src/objects.cc M src/runtime.cc M src/string-stream.cc M src/stub-cache.h M src/utils.h M src/utils.cc M src/x64/codegen-x64.cc M src/x64/ic-x64.cc M src/x64/stub-cache-x64.cc M test/cctest/test-alloc.cc M test/cctest/test-api.cc M test/cctest/test-heap.cc --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
