OLD | NEW |
1 // Copyright 2011 the V8 project authors. All rights reserved. | 1 // Copyright 2014 the V8 project authors. All rights reserved. |
2 // Redistribution and use in source and binary forms, with or without | 2 // Redistribution and use in source and binary forms, with or without |
3 // modification, are permitted provided that the following conditions are | 3 // modification, are permitted provided that the following conditions are |
4 // met: | 4 // met: |
5 // | 5 // |
6 // * Redistributions of source code must retain the above copyright | 6 // * Redistributions of source code must retain the above copyright |
7 // notice, this list of conditions and the following disclaimer. | 7 // notice, this list of conditions and the following disclaimer. |
8 // * Redistributions in binary form must reproduce the above | 8 // * Redistributions in binary form must reproduce the above |
9 // copyright notice, this list of conditions and the following | 9 // copyright notice, this list of conditions and the following |
10 // disclaimer in the documentation and/or other materials provided | 10 // disclaimer in the documentation and/or other materials provided |
11 // with the distribution. | 11 // with the distribution. |
12 // * Neither the name of Google Inc. nor the names of its | 12 // * Neither the name of Google Inc. nor the names of its |
13 // contributors may be used to endorse or promote products derived | 13 // contributors may be used to endorse or promote products derived |
14 // from this software without specific prior written permission. | 14 // from this software without specific prior written permission. |
15 // | 15 // |
16 // THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS | 16 // THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS |
17 // "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT | 17 // "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT |
18 // LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR | 18 // LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR |
19 // A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT | 19 // A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT |
20 // OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, | 20 // OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, |
21 // SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT | 21 // SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT |
22 // LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, | 22 // LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, |
23 // DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY | 23 // DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY |
24 // THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT | 24 // THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT |
25 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE | 25 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE |
26 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. | 26 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. |
27 | 27 |
| 28 #include <stdlib.h> |
| 29 |
28 #include "v8.h" | 30 #include "v8.h" |
29 | 31 |
30 #include "api.h" | 32 #include "cctest.h" |
31 #include "debug.h" | |
32 #include "execution.h" | |
33 #include "factory.h" | 33 #include "factory.h" |
34 #include "macro-assembler.h" | 34 |
35 #include "objects.h" | 35 #include "close-table.h" |
36 #include "global-handles.h" | 36 |
37 #include "cctest.h" | 37 namespace { |
38 | 38 |
39 using namespace v8::internal; | 39 using namespace v8::internal; |
40 | 40 |
| 41 TEST(Set) { |
| 42 LocalContext context; |
| 43 Isolate* isolate = CcTest::i_isolate(); |
| 44 Factory* factory = isolate->factory(); |
| 45 HandleScope scope(isolate); |
| 46 Handle<CloseTableSet> close_set = factory->NewCloseTableSet(); |
| 47 CHECK_EQ(2, close_set->NumberOfBuckets()); |
| 48 CHECK_EQ(0, close_set->NumberOfElements()); |
| 49 CHECK_EQ(0, close_set->NumberOfDeletedElements()); |
| 50 |
| 51 Handle<Map> map = factory->NewMap(JS_OBJECT_TYPE, JSObject::kHeaderSize); |
| 52 Handle<JSObject> obj = factory->NewJSObjectFromMap(map); |
| 53 CHECK(!close_set->Contains(*obj)); |
| 54 close_set = CloseTableSet::Add(close_set, obj); |
| 55 CHECK_EQ(1, close_set->NumberOfElements()); |
| 56 CHECK(close_set->Contains(*obj)); |
| 57 close_set = CloseTableSet::Remove(close_set, obj); |
| 58 CHECK_EQ(0, close_set->NumberOfElements()); |
| 59 CHECK(!close_set->Contains(*obj)); |
| 60 |
| 61 // Test for collisions/chaining |
| 62 Handle<JSObject> obj1 = factory->NewJSObjectFromMap(map); |
| 63 close_set = CloseTableSet::Add(close_set, obj1); |
| 64 Handle<JSObject> obj2 = factory->NewJSObjectFromMap(map); |
| 65 close_set = CloseTableSet::Add(close_set, obj2); |
| 66 Handle<JSObject> obj3 = factory->NewJSObjectFromMap(map); |
| 67 close_set = CloseTableSet::Add(close_set, obj3); |
| 68 CHECK_EQ(3, close_set->NumberOfElements()); |
| 69 CHECK(close_set->Contains(*obj1)); |
| 70 CHECK(close_set->Contains(*obj2)); |
| 71 CHECK(close_set->Contains(*obj3)); |
| 72 |
| 73 // Test growth |
| 74 close_set = CloseTableSet::Add(close_set, obj); |
| 75 Handle<JSObject> obj4 = factory->NewJSObjectFromMap(map); |
| 76 close_set = CloseTableSet::Add(close_set, obj4); |
| 77 CHECK(close_set->Contains(*obj)); |
| 78 CHECK(close_set->Contains(*obj1)); |
| 79 CHECK(close_set->Contains(*obj2)); |
| 80 CHECK(close_set->Contains(*obj3)); |
| 81 CHECK(close_set->Contains(*obj4)); |
| 82 CHECK_EQ(5, close_set->NumberOfElements()); |
| 83 CHECK_EQ(0, close_set->NumberOfDeletedElements()); |
| 84 CHECK_EQ(4, close_set->NumberOfBuckets()); |
| 85 |
| 86 // Test shrinking |
| 87 close_set = CloseTableSet::Remove(close_set, obj); |
| 88 close_set = CloseTableSet::Remove(close_set, obj1); |
| 89 close_set = CloseTableSet::Remove(close_set, obj2); |
| 90 close_set = CloseTableSet::Remove(close_set, obj3); |
| 91 CHECK_EQ(1, close_set->NumberOfElements()); |
| 92 CHECK_EQ(2, close_set->NumberOfBuckets()); |
| 93 } |
| 94 |
| 95 TEST(Map) { |
| 96 LocalContext context; |
| 97 Isolate* isolate = CcTest::i_isolate(); |
| 98 Factory* factory = isolate->factory(); |
| 99 HandleScope scope(isolate); |
| 100 Handle<CloseTableMap> close_map = factory->NewCloseTableMap(); |
| 101 CHECK_EQ(2, close_map->NumberOfBuckets()); |
| 102 CHECK_EQ(0, close_map->NumberOfElements()); |
| 103 CHECK_EQ(0, close_map->NumberOfDeletedElements()); |
| 104 |
| 105 Handle<Map> map = factory->NewMap(JS_OBJECT_TYPE, JSObject::kHeaderSize); |
| 106 Handle<JSObject> obj = factory->NewJSObjectFromMap(map); |
| 107 Handle<JSObject> val = factory->NewJSObjectFromMap(map); |
| 108 CHECK(close_map->Lookup(*obj)->IsTheHole()); |
| 109 close_map = CloseTableMap::Put(close_map, obj, val); |
| 110 CHECK_EQ(1, close_map->NumberOfElements()); |
| 111 CHECK(close_map->Lookup(*obj)->SameValue(*val)); |
| 112 close_map = CloseTableMap::Put(close_map, obj, factory->the_hole_value()); |
| 113 CHECK_EQ(0, close_map->NumberOfElements()); |
| 114 CHECK(close_map->Lookup(*obj)->IsTheHole()); |
| 115 |
| 116 // Test for collisions/chaining |
| 117 Handle<JSObject> obj1 = factory->NewJSObjectFromMap(map); |
| 118 Handle<JSObject> obj2 = factory->NewJSObjectFromMap(map); |
| 119 Handle<JSObject> obj3 = factory->NewJSObjectFromMap(map); |
| 120 Handle<JSObject> val1 = factory->NewJSObjectFromMap(map); |
| 121 Handle<JSObject> val2 = factory->NewJSObjectFromMap(map); |
| 122 Handle<JSObject> val3 = factory->NewJSObjectFromMap(map); |
| 123 close_map = CloseTableMap::Put(close_map, obj1, val1); |
| 124 close_map = CloseTableMap::Put(close_map, obj2, val2); |
| 125 close_map = CloseTableMap::Put(close_map, obj3, val3); |
| 126 CHECK_EQ(3, close_map->NumberOfElements()); |
| 127 CHECK(close_map->Lookup(*obj1)->SameValue(*val1)); |
| 128 CHECK(close_map->Lookup(*obj2)->SameValue(*val2)); |
| 129 CHECK(close_map->Lookup(*obj3)->SameValue(*val3)); |
| 130 |
| 131 // Test growth |
| 132 close_map = CloseTableMap::Put(close_map, obj, val); |
| 133 Handle<JSObject> obj4 = factory->NewJSObjectFromMap(map); |
| 134 Handle<JSObject> val4 = factory->NewJSObjectFromMap(map); |
| 135 close_map = CloseTableMap::Put(close_map, obj4, val4); |
| 136 CHECK(close_map->Lookup(*obj)->SameValue(*val)); |
| 137 CHECK(close_map->Lookup(*obj1)->SameValue(*val1)); |
| 138 CHECK(close_map->Lookup(*obj2)->SameValue(*val2)); |
| 139 CHECK(close_map->Lookup(*obj3)->SameValue(*val3)); |
| 140 CHECK(close_map->Lookup(*obj4)->SameValue(*val4)); |
| 141 CHECK_EQ(5, close_map->NumberOfElements()); |
| 142 CHECK_EQ(4, close_map->NumberOfBuckets()); |
| 143 |
| 144 // Test shrinking |
| 145 close_map = CloseTableMap::Put(close_map, obj, factory->the_hole_value()); |
| 146 close_map = CloseTableMap::Put(close_map, obj1, factory->the_hole_value()); |
| 147 close_map = CloseTableMap::Put(close_map, obj2, factory->the_hole_value()); |
| 148 close_map = CloseTableMap::Put(close_map, obj3, factory->the_hole_value()); |
| 149 CHECK_EQ(1, close_map->NumberOfElements()); |
| 150 CHECK_EQ(2, close_map->NumberOfBuckets()); |
| 151 } |
| 152 |
| 153 // --------------------------------------------------------- |
| 154 // BELOW THIS LINE TESTS WERE CONVERTED FROM test-dictionary |
| 155 // --------------------------------------------------------- |
41 | 156 |
42 TEST(ObjectHashTable) { | 157 TEST(ObjectHashTable) { |
43 LocalContext context; | 158 LocalContext context; |
44 Isolate* isolate = CcTest::i_isolate(); | 159 Isolate* isolate = CcTest::i_isolate(); |
45 Factory* factory = isolate->factory(); | 160 Factory* factory = isolate->factory(); |
46 v8::HandleScope scope(context->GetIsolate()); | 161 v8::HandleScope scope(context->GetIsolate()); |
47 Handle<ObjectHashTable> table = factory->NewObjectHashTable(23); | 162 Handle<CloseTableMap> table = factory->NewCloseTableMap(); |
48 Handle<JSObject> a = factory->NewJSArray(7); | 163 Handle<JSObject> a = factory->NewJSArray(7); |
49 Handle<JSObject> b = factory->NewJSArray(11); | 164 Handle<JSObject> b = factory->NewJSArray(11); |
50 table = ObjectHashTable::Put(table, a, b); | 165 table = CloseTableMap::Put(table, a, b); |
51 CHECK_EQ(table->NumberOfElements(), 1); | 166 CHECK_EQ(table->NumberOfElements(), 1); |
52 CHECK_EQ(table->Lookup(*a), *b); | 167 CHECK_EQ(table->Lookup(*a), *b); |
53 CHECK_EQ(table->Lookup(*b), CcTest::heap()->the_hole_value()); | 168 CHECK_EQ(table->Lookup(*b), CcTest::heap()->the_hole_value()); |
54 | 169 |
55 // Keys still have to be valid after objects were moved. | 170 // Keys still have to be valid after objects were moved. |
56 CcTest::heap()->CollectGarbage(NEW_SPACE); | 171 CcTest::heap()->CollectGarbage(NEW_SPACE); |
57 CHECK_EQ(table->NumberOfElements(), 1); | 172 CHECK_EQ(table->NumberOfElements(), 1); |
58 CHECK_EQ(table->Lookup(*a), *b); | 173 CHECK_EQ(table->Lookup(*a), *b); |
59 CHECK_EQ(table->Lookup(*b), CcTest::heap()->the_hole_value()); | 174 CHECK_EQ(table->Lookup(*b), CcTest::heap()->the_hole_value()); |
60 | 175 |
61 // Keys that are overwritten should not change number of elements. | 176 // Keys that are overwritten should not change number of elements. |
62 table = ObjectHashTable::Put(table, a, factory->NewJSArray(13)); | 177 table = CloseTableMap::Put(table, a, factory->NewJSArray(13)); |
63 CHECK_EQ(table->NumberOfElements(), 1); | 178 CHECK_EQ(table->NumberOfElements(), 1); |
64 CHECK_NE(table->Lookup(*a), *b); | 179 CHECK_NE(table->Lookup(*a), *b); |
65 | 180 |
66 // Keys mapped to the hole should be removed permanently. | 181 // Keys mapped to the hole should be removed permanently. |
67 table = ObjectHashTable::Put(table, a, factory->the_hole_value()); | 182 table = CloseTableMap::Put(table, a, factory->the_hole_value()); |
68 CHECK_EQ(table->NumberOfElements(), 0); | 183 CHECK_EQ(table->NumberOfElements(), 0); |
69 CHECK_EQ(table->NumberOfDeletedElements(), 1); | 184 //CHECK_EQ(table->NumberOfDeletedElements(), 1); |
70 CHECK_EQ(table->Lookup(*a), CcTest::heap()->the_hole_value()); | 185 CHECK_EQ(table->Lookup(*a), CcTest::heap()->the_hole_value()); |
71 | 186 |
72 // Keys should map back to their respective values and also should get | 187 // Keys should map back to their respective values and also should get |
73 // an identity hash code generated. | 188 // an identity hash code generated. |
74 for (int i = 0; i < 100; i++) { | 189 for (int i = 0; i < 100; i++) { |
75 Handle<JSReceiver> key = factory->NewJSArray(7); | 190 Handle<JSReceiver> key = factory->NewJSArray(7); |
76 Handle<JSObject> value = factory->NewJSArray(11); | 191 Handle<JSObject> value = factory->NewJSArray(11); |
77 table = ObjectHashTable::Put(table, key, value); | 192 table = CloseTableMap::Put(table, key, value); |
78 CHECK_EQ(table->NumberOfElements(), i + 1); | 193 CHECK_EQ(table->NumberOfElements(), i + 1); |
79 CHECK_NE(table->FindEntry(*key), ObjectHashTable::kNotFound); | 194 CHECK_NE(table->FindEntry(*key), CloseTableMap::kNotFound); |
80 CHECK_EQ(table->Lookup(*key), *value); | 195 CHECK_EQ(table->Lookup(*key), *value); |
81 CHECK(key->GetIdentityHash()->IsSmi()); | 196 CHECK(key->GetIdentityHash()->IsSmi()); |
82 } | 197 } |
83 | 198 |
84 // Keys never added to the map which already have an identity hash | 199 // Keys never added to the map which already have an identity hash |
85 // code should not be found. | 200 // code should not be found. |
86 for (int i = 0; i < 100; i++) { | 201 for (int i = 0; i < 100; i++) { |
87 Handle<JSReceiver> key = factory->NewJSArray(7); | 202 Handle<JSReceiver> key = factory->NewJSArray(7); |
88 CHECK(JSReceiver::GetOrCreateIdentityHash(key)->IsSmi()); | 203 CHECK(JSReceiver::GetOrCreateIdentityHash(key)->IsSmi()); |
89 CHECK_EQ(table->FindEntry(*key), ObjectHashTable::kNotFound); | 204 CHECK_EQ(table->FindEntry(*key), CloseTableMap::kNotFound); |
90 CHECK_EQ(table->Lookup(*key), CcTest::heap()->the_hole_value()); | 205 CHECK_EQ(table->Lookup(*key), CcTest::heap()->the_hole_value()); |
91 CHECK(key->GetIdentityHash()->IsSmi()); | 206 CHECK(key->GetIdentityHash()->IsSmi()); |
92 } | 207 } |
93 | 208 |
94 // Keys that don't have an identity hash should not be found and also | 209 // Keys that don't have an identity hash should not be found and also |
95 // should not get an identity hash code generated. | 210 // should not get an identity hash code generated. |
96 for (int i = 0; i < 100; i++) { | 211 for (int i = 0; i < 100; i++) { |
97 Handle<JSReceiver> key = factory->NewJSArray(7); | 212 Handle<JSReceiver> key = factory->NewJSArray(7); |
98 CHECK_EQ(table->Lookup(*key), CcTest::heap()->the_hole_value()); | 213 CHECK_EQ(table->Lookup(*key), CcTest::heap()->the_hole_value()); |
99 CHECK_EQ(key->GetIdentityHash(), | 214 CHECK_EQ(key->GetIdentityHash(), |
100 CcTest::heap()->undefined_value()); | 215 CcTest::heap()->undefined_value()); |
101 } | 216 } |
102 } | 217 } |
103 | 218 |
104 | 219 |
105 class ObjectHashTableTest: public ObjectHashTable { | |
106 public: | |
107 void insert(int entry, int key, int value) { | |
108 set(EntryToIndex(entry), Smi::FromInt(key)); | |
109 set(EntryToIndex(entry) + 1, Smi::FromInt(value)); | |
110 } | |
111 | |
112 int lookup(int key) { | |
113 return Smi::cast(Lookup(Smi::FromInt(key)))->value(); | |
114 } | |
115 | |
116 int capacity() { | |
117 return Capacity(); | |
118 } | |
119 }; | |
120 | |
121 | |
122 TEST(HashTableRehash) { | |
123 LocalContext context; | |
124 Isolate* isolate = CcTest::i_isolate(); | |
125 Factory* factory = isolate->factory(); | |
126 v8::HandleScope scope(context->GetIsolate()); | |
127 // Test almost filled table. | |
128 { | |
129 Handle<ObjectHashTable> table = factory->NewObjectHashTable(100); | |
130 ObjectHashTableTest* t = reinterpret_cast<ObjectHashTableTest*>(*table); | |
131 int capacity = t->capacity(); | |
132 for (int i = 0; i < capacity - 1; i++) { | |
133 t->insert(i, i * i, i); | |
134 } | |
135 t->Rehash(Smi::FromInt(0)); | |
136 for (int i = 0; i < capacity - 1; i++) { | |
137 CHECK_EQ(i, t->lookup(i * i)); | |
138 } | |
139 } | |
140 // Test half-filled table. | |
141 { | |
142 Handle<ObjectHashTable> table = factory->NewObjectHashTable(100); | |
143 ObjectHashTableTest* t = reinterpret_cast<ObjectHashTableTest*>(*table); | |
144 int capacity = t->capacity(); | |
145 for (int i = 0; i < capacity / 2; i++) { | |
146 t->insert(i, i * i, i); | |
147 } | |
148 t->Rehash(Smi::FromInt(0)); | |
149 for (int i = 0; i < capacity / 2; i++) { | |
150 CHECK_EQ(i, t->lookup(i * i)); | |
151 } | |
152 } | |
153 } | |
154 | |
155 | |
156 #ifdef DEBUG | 220 #ifdef DEBUG |
157 TEST(ObjectHashSetCausesGC) { | 221 TEST(CloseTableSetCausesGC) { |
158 i::FLAG_stress_compaction = false; | 222 i::FLAG_stress_compaction = false; |
159 LocalContext context; | 223 LocalContext context; |
160 Isolate* isolate = CcTest::i_isolate(); | 224 Isolate* isolate = CcTest::i_isolate(); |
161 Factory* factory = isolate->factory(); | 225 Factory* factory = isolate->factory(); |
162 v8::HandleScope scope(context->GetIsolate()); | 226 v8::HandleScope scope(context->GetIsolate()); |
163 Handle<ObjectHashSet> table = factory->NewObjectHashSet(1); | 227 Handle<CloseTableSet> table = factory->NewCloseTableSet(); |
164 Handle<JSObject> key = factory->NewJSArray(0); | 228 Handle<JSObject> key = factory->NewJSArray(0); |
165 v8::Handle<v8::Object> key_obj = v8::Utils::ToLocal(key); | 229 v8::Handle<v8::Object> key_obj = v8::Utils::ToLocal(key); |
166 | 230 |
167 // Force allocation of hash table backing store for hidden properties. | 231 // Force allocation of hash table backing store for hidden properties. |
168 key_obj->SetHiddenValue(v8_str("key 1"), v8_str("val 1")); | 232 key_obj->SetHiddenValue(v8_str("key 1"), v8_str("val 1")); |
169 key_obj->SetHiddenValue(v8_str("key 2"), v8_str("val 2")); | 233 key_obj->SetHiddenValue(v8_str("key 2"), v8_str("val 2")); |
170 key_obj->SetHiddenValue(v8_str("key 3"), v8_str("val 3")); | 234 key_obj->SetHiddenValue(v8_str("key 3"), v8_str("val 3")); |
171 | 235 |
172 // Simulate a full heap so that generating an identity hash code | 236 // Simulate a full heap so that generating an identity hash code |
173 // in subsequent calls will request GC. | 237 // in subsequent calls will request GC. |
174 SimulateFullSpace(CcTest::heap()->new_space()); | 238 SimulateFullSpace(CcTest::heap()->new_space()); |
175 SimulateFullSpace(CcTest::heap()->old_pointer_space()); | 239 SimulateFullSpace(CcTest::heap()->old_pointer_space()); |
176 | 240 |
177 // Calling Contains() should not cause GC ever. | 241 // Calling Contains() should not cause GC ever. |
178 int gc_count = isolate->heap()->gc_count(); | 242 int gc_count = isolate->heap()->gc_count(); |
179 CHECK(!table->Contains(*key)); | 243 CHECK(!table->Contains(*key)); |
180 CHECK(gc_count == isolate->heap()->gc_count()); | 244 CHECK(gc_count == isolate->heap()->gc_count()); |
181 | 245 |
182 // Calling Remove() will not cause GC in this case. | 246 // Calling Remove() will not cause GC in this case. |
183 table = ObjectHashSet::Remove(table, key); | 247 table = CloseTableSet::Remove(table, key); |
184 CHECK(gc_count == isolate->heap()->gc_count()); | 248 CHECK(gc_count == isolate->heap()->gc_count()); |
185 | 249 |
186 // Calling Add() should cause GC. | 250 // Calling Add() should cause GC. |
187 table = ObjectHashSet::Add(table, key); | 251 table = CloseTableSet::Add(table, key); |
188 CHECK(gc_count < isolate->heap()->gc_count()); | 252 CHECK(gc_count < isolate->heap()->gc_count()); |
189 } | 253 } |
190 #endif | 254 #endif |
191 | 255 |
192 | 256 |
193 #ifdef DEBUG | 257 #ifdef DEBUG |
194 TEST(ObjectHashTableCausesGC) { | 258 TEST(CloseTableMapCausesGC) { |
195 i::FLAG_stress_compaction = false; | 259 i::FLAG_stress_compaction = false; |
196 LocalContext context; | 260 LocalContext context; |
197 Isolate* isolate = CcTest::i_isolate(); | 261 Isolate* isolate = CcTest::i_isolate(); |
198 Factory* factory = isolate->factory(); | 262 Factory* factory = isolate->factory(); |
199 v8::HandleScope scope(context->GetIsolate()); | 263 v8::HandleScope scope(context->GetIsolate()); |
200 Handle<ObjectHashTable> table = factory->NewObjectHashTable(1); | 264 Handle<CloseTableMap> table = factory->NewCloseTableMap(); |
201 Handle<JSObject> key = factory->NewJSArray(0); | 265 Handle<JSObject> key = factory->NewJSArray(0); |
202 v8::Handle<v8::Object> key_obj = v8::Utils::ToLocal(key); | 266 v8::Handle<v8::Object> key_obj = v8::Utils::ToLocal(key); |
203 | 267 |
204 // Force allocation of hash table backing store for hidden properties. | 268 // Force allocation of hash table backing store for hidden properties. |
205 key_obj->SetHiddenValue(v8_str("key 1"), v8_str("val 1")); | 269 key_obj->SetHiddenValue(v8_str("key 1"), v8_str("val 1")); |
206 key_obj->SetHiddenValue(v8_str("key 2"), v8_str("val 2")); | 270 key_obj->SetHiddenValue(v8_str("key 2"), v8_str("val 2")); |
207 key_obj->SetHiddenValue(v8_str("key 3"), v8_str("val 3")); | 271 key_obj->SetHiddenValue(v8_str("key 3"), v8_str("val 3")); |
208 | 272 |
209 // Simulate a full heap so that generating an identity hash code | 273 // Simulate a full heap so that generating an identity hash code |
210 // in subsequent calls will request GC. | 274 // in subsequent calls will request GC. |
211 SimulateFullSpace(CcTest::heap()->new_space()); | 275 SimulateFullSpace(CcTest::heap()->new_space()); |
212 SimulateFullSpace(CcTest::heap()->old_pointer_space()); | 276 SimulateFullSpace(CcTest::heap()->old_pointer_space()); |
213 | 277 |
214 // Calling Lookup() should not cause GC ever. | 278 // Calling Lookup() should not cause GC ever. |
215 CHECK(table->Lookup(*key)->IsTheHole()); | 279 CHECK(table->Lookup(*key)->IsTheHole()); |
216 | 280 |
217 // Calling Put() should request GC by returning a failure. | 281 // Calling Put() should request GC by returning a failure. |
218 int gc_count = isolate->heap()->gc_count(); | 282 int gc_count = isolate->heap()->gc_count(); |
219 ObjectHashTable::Put(table, key, key); | 283 CloseTableMap::Put(table, key, key); |
220 CHECK(gc_count < isolate->heap()->gc_count()); | 284 CHECK(gc_count < isolate->heap()->gc_count()); |
221 } | 285 } |
222 #endif | 286 #endif |
| 287 |
| 288 } |
OLD | NEW |