1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

deps: cherry-pick r21466 from v8 trunk

Browse Source 
Check for cached transition to ExternalArray elements kind.
See [1] and [2] for details.

[1] https://code.google.com/p/v8/issues/detail?id=3337
[2] https://codereview.chromium.org/291193011

Signed-off-by: Fedor Indutny <fedor@indutny.com>
This commit is contained in:
Ben Noordhuis 2014-05-23 14:01:17 +00:00 committed by Fedor Indutny
parent 5d178188a7
commit d78a3787d2
2 changed files with 25 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
deps/v8/src/objects.cc vendored
View File

@ -3362,9 +3362,16 @@ static Map* FindClosestElementsTransition(Map* map, ElementsKind to_kind) {
? to_kind
: TERMINAL_FAST_ELEMENTS_KIND;
// Support for legacy API.
// Support for legacy API: SetIndexedPropertiesTo{External,Pixel}Data
// allows to change elements from arbitrary kind to any ExternalArray
// elements kind. Satisfy its requirements, checking whether we already
// have the cached transition.
if (IsExternalArrayElementsKind(to_kind) &&
!IsFixedTypedArrayElementsKind(map->elements_kind())) {
if (map->HasElementsTransition()) {
Map* next_map = map->elements_transition_map();
if (next_map->elements_kind() == to_kind) return next_map;
}
return map;
}

17
deps/v8/test/cctest/test-api.cc vendored
View File

@ -21341,6 +21341,23 @@ THREADED_TEST(Regress142088) {
}
THREADED_TEST(Regress3337) {
LocalContext context;
v8::Isolate* isolate = context->GetIsolate();
v8::HandleScope scope(isolate);
Local<v8::Object> o1 = Object::New(isolate);
Local<v8::Object> o2 = Object::New(isolate);
i::Handle<i::JSObject> io1 = v8::Utils::OpenHandle(*o1);
i::Handle<i::JSObject> io2 = v8::Utils::OpenHandle(*o2);
CHECK(io1->map() == io2->map());
o1->SetIndexedPropertiesToExternalArrayData(
NULL, v8::kExternalUint32Array, 0);
o2->SetIndexedPropertiesToExternalArrayData(
NULL, v8::kExternalUint32Array, 0);
CHECK(io1->map() == io2->map());
}
THREADED_TEST(Regress137496) {
i::FLAG_expose_gc = true;
LocalContext context;