Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(144)

Side by Side Diff: src/ic/mips64/stub-cache-mips64.cc

Issue 2123983004: [ic] Split megamorphic stub cache in two caches (for loads and for stores). (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@flags-fix
Patch Set: Rebasing Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #if V8_TARGET_ARCH_MIPS64 5 #if V8_TARGET_ARCH_MIPS64
6 6
7 #include "src/codegen.h" 7 #include "src/codegen.h"
8 #include "src/ic/ic.h" 8 #include "src/ic/ic.h"
9 #include "src/ic/stub-cache.h" 9 #include "src/ic/stub-cache.h"
10 #include "src/interface-descriptors.h" 10 #include "src/interface-descriptors.h"
11 11
12 namespace v8 { 12 namespace v8 {
13 namespace internal { 13 namespace internal {
14 14
15 #define __ ACCESS_MASM(masm) 15 #define __ ACCESS_MASM(masm)
16 16
17 static void ProbeTable(Isolate* isolate, MacroAssembler* masm, 17 static void ProbeTable(StubCache* stub_cache, MacroAssembler* masm,
18 Code::Flags flags, StubCache::Table table, 18 Code::Flags flags, StubCache::Table table,
19 Register receiver, Register name, 19 Register receiver, Register name,
20 // Number of the cache entry, not scaled. 20 // Number of the cache entry, not scaled.
21 Register offset, Register scratch, Register scratch2, 21 Register offset, Register scratch, Register scratch2,
22 Register offset_scratch) { 22 Register offset_scratch) {
23 ExternalReference key_offset(isolate->stub_cache()->key_reference(table)); 23 ExternalReference key_offset(stub_cache->key_reference(table));
24 ExternalReference value_offset(isolate->stub_cache()->value_reference(table)); 24 ExternalReference value_offset(stub_cache->value_reference(table));
25 ExternalReference map_offset(isolate->stub_cache()->map_reference(table)); 25 ExternalReference map_offset(stub_cache->map_reference(table));
26 26
27 uint64_t key_off_addr = reinterpret_cast<uint64_t>(key_offset.address()); 27 uint64_t key_off_addr = reinterpret_cast<uint64_t>(key_offset.address());
28 uint64_t value_off_addr = reinterpret_cast<uint64_t>(value_offset.address()); 28 uint64_t value_off_addr = reinterpret_cast<uint64_t>(value_offset.address());
29 uint64_t map_off_addr = reinterpret_cast<uint64_t>(map_offset.address()); 29 uint64_t map_off_addr = reinterpret_cast<uint64_t>(map_offset.address());
30 30
31 // Check the relative positions of the address fields. 31 // Check the relative positions of the address fields.
32 DCHECK(value_off_addr > key_off_addr); 32 DCHECK(value_off_addr > key_off_addr);
33 DCHECK((value_off_addr - key_off_addr) % 4 == 0); 33 DCHECK((value_off_addr - key_off_addr) % 4 == 0);
34 DCHECK((value_off_addr - key_off_addr) < (256 * 4)); 34 DCHECK((value_off_addr - key_off_addr) < (256 * 4));
35 DCHECK(map_off_addr > key_off_addr); 35 DCHECK(map_off_addr > key_off_addr);
(...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after
79 #endif 79 #endif
80 80
81 // Jump to the first instruction in the code stub. 81 // Jump to the first instruction in the code stub.
82 __ Daddu(at, code, Operand(Code::kHeaderSize - kHeapObjectTag)); 82 __ Daddu(at, code, Operand(Code::kHeaderSize - kHeapObjectTag));
83 __ Jump(at); 83 __ Jump(at);
84 84
85 // Miss: fall through. 85 // Miss: fall through.
86 __ bind(&miss); 86 __ bind(&miss);
87 } 87 }
88 88
89 89 void StubCache::GenerateProbe(MacroAssembler* masm, Register receiver,
90 void StubCache::GenerateProbe(MacroAssembler* masm, Code::Kind ic_kind,
91 Code::Flags flags, Register receiver,
92 Register name, Register scratch, Register extra, 90 Register name, Register scratch, Register extra,
93 Register extra2, Register extra3) { 91 Register extra2, Register extra3) {
94 Isolate* isolate = masm->isolate(); 92 Code::Flags flags =
93 Code::RemoveHolderFromFlags(Code::ComputeHandlerFlags(ic_kind_));
94
95 Label miss; 95 Label miss;
96 96
97 // Make sure that code is valid. The multiplying code relies on the 97 // Make sure that code is valid. The multiplying code relies on the
98 // entry size being 12. 98 // entry size being 12.
99 // DCHECK(sizeof(Entry) == 12); 99 // DCHECK(sizeof(Entry) == 12);
100 // DCHECK(sizeof(Entry) == 3 * kPointerSize); 100 // DCHECK(sizeof(Entry) == 3 * kPointerSize);
101 101
102 // Make sure that there are no register conflicts. 102 // Make sure that there are no register conflicts.
103 DCHECK(!AreAliased(receiver, name, scratch, extra, extra2, extra3)); 103 DCHECK(!AreAliased(receiver, name, scratch, extra, extra2, extra3));
104 104
105 // Check register validity. 105 // Check register validity.
106 DCHECK(!scratch.is(no_reg)); 106 DCHECK(!scratch.is(no_reg));
107 DCHECK(!extra.is(no_reg)); 107 DCHECK(!extra.is(no_reg));
108 DCHECK(!extra2.is(no_reg)); 108 DCHECK(!extra2.is(no_reg));
109 DCHECK(!extra3.is(no_reg)); 109 DCHECK(!extra3.is(no_reg));
110 110
111 #ifdef DEBUG 111 #ifdef DEBUG
112 // If vector-based ics are in use, ensure that scratch, extra, extra2 and 112 // If vector-based ics are in use, ensure that scratch, extra, extra2 and
113 // extra3 don't conflict with the vector and slot registers, which need 113 // extra3 don't conflict with the vector and slot registers, which need
114 // to be preserved for a handler call or miss. 114 // to be preserved for a handler call or miss.
115 if (IC::ICUseVector(ic_kind)) { 115 if (IC::ICUseVector(ic_kind_)) {
116 Register vector, slot; 116 Register vector, slot;
117 if (ic_kind == Code::STORE_IC || ic_kind == Code::KEYED_STORE_IC) { 117 if (ic_kind_ == Code::STORE_IC || ic_kind_ == Code::KEYED_STORE_IC) {
118 vector = VectorStoreICDescriptor::VectorRegister(); 118 vector = VectorStoreICDescriptor::VectorRegister();
119 slot = VectorStoreICDescriptor::SlotRegister(); 119 slot = VectorStoreICDescriptor::SlotRegister();
120 } else { 120 } else {
121 DCHECK(ic_kind_ == Code::LOAD_IC || ic_kind_ == Code::KEYED_LOAD_IC);
121 vector = LoadWithVectorDescriptor::VectorRegister(); 122 vector = LoadWithVectorDescriptor::VectorRegister();
122 slot = LoadWithVectorDescriptor::SlotRegister(); 123 slot = LoadWithVectorDescriptor::SlotRegister();
123 } 124 }
124 DCHECK(!AreAliased(vector, slot, scratch, extra, extra2, extra3)); 125 DCHECK(!AreAliased(vector, slot, scratch, extra, extra2, extra3));
125 } 126 }
126 #endif 127 #endif
127 128
128 Counters* counters = masm->isolate()->counters(); 129 Counters* counters = masm->isolate()->counters();
129 __ IncrementCounter(counters->megamorphic_stub_cache_probes(), 1, extra2, 130 __ IncrementCounter(counters->megamorphic_stub_cache_probes(), 1, extra2,
130 extra3); 131 extra3);
131 132
132 // Check that the receiver isn't a smi. 133 // Check that the receiver isn't a smi.
133 __ JumpIfSmi(receiver, &miss); 134 __ JumpIfSmi(receiver, &miss);
134 135
135 // Get the map of the receiver and compute the hash. 136 // Get the map of the receiver and compute the hash.
136 __ ld(scratch, FieldMemOperand(name, Name::kHashFieldOffset)); 137 __ ld(scratch, FieldMemOperand(name, Name::kHashFieldOffset));
137 __ ld(at, FieldMemOperand(receiver, HeapObject::kMapOffset)); 138 __ ld(at, FieldMemOperand(receiver, HeapObject::kMapOffset));
138 __ Daddu(scratch, scratch, at); 139 __ Daddu(scratch, scratch, at);
139 uint64_t mask = kPrimaryTableSize - 1; 140 uint64_t mask = kPrimaryTableSize - 1;
140 // We shift out the last two bits because they are not part of the hash and 141 // We shift out the last two bits because they are not part of the hash and
141 // they are always 01 for maps. 142 // they are always 01 for maps.
142 __ dsrl(scratch, scratch, kCacheIndexShift); 143 __ dsrl(scratch, scratch, kCacheIndexShift);
143 __ Xor(scratch, scratch, Operand((flags >> kCacheIndexShift) & mask)); 144 __ Xor(scratch, scratch, Operand((flags >> kCacheIndexShift) & mask));
144 __ And(scratch, scratch, Operand(mask)); 145 __ And(scratch, scratch, Operand(mask));
145 146
146 // Probe the primary table. 147 // Probe the primary table.
147 ProbeTable(isolate, masm, flags, kPrimary, receiver, name, scratch, extra, 148 ProbeTable(this, masm, flags, kPrimary, receiver, name, scratch, extra,
148 extra2, extra3); 149 extra2, extra3);
149 150
150 // Primary miss: Compute hash for secondary probe. 151 // Primary miss: Compute hash for secondary probe.
151 __ dsrl(at, name, kCacheIndexShift); 152 __ dsrl(at, name, kCacheIndexShift);
152 __ Dsubu(scratch, scratch, at); 153 __ Dsubu(scratch, scratch, at);
153 uint64_t mask2 = kSecondaryTableSize - 1; 154 uint64_t mask2 = kSecondaryTableSize - 1;
154 __ Daddu(scratch, scratch, Operand((flags >> kCacheIndexShift) & mask2)); 155 __ Daddu(scratch, scratch, Operand((flags >> kCacheIndexShift) & mask2));
155 __ And(scratch, scratch, Operand(mask2)); 156 __ And(scratch, scratch, Operand(mask2));
156 157
157 // Probe the secondary table. 158 // Probe the secondary table.
158 ProbeTable(isolate, masm, flags, kSecondary, receiver, name, scratch, extra, 159 ProbeTable(this, masm, flags, kSecondary, receiver, name, scratch, extra,
159 extra2, extra3); 160 extra2, extra3);
160 161
161 // Cache miss: Fall-through and let caller handle the miss by 162 // Cache miss: Fall-through and let caller handle the miss by
162 // entering the runtime system. 163 // entering the runtime system.
163 __ bind(&miss); 164 __ bind(&miss);
164 __ IncrementCounter(counters->megamorphic_stub_cache_misses(), 1, extra2, 165 __ IncrementCounter(counters->megamorphic_stub_cache_misses(), 1, extra2,
165 extra3); 166 extra3);
166 } 167 }
167 168
168 169
169 #undef __ 170 #undef __
170 } // namespace internal 171 } // namespace internal
171 } // namespace v8 172 } // namespace v8
172 173
173 #endif // V8_TARGET_ARCH_MIPS64 174 #endif // V8_TARGET_ARCH_MIPS64
OLDNEW
« no previous file with comments | « src/ic/mips64/ic-mips64.cc ('k') | src/ic/ppc/ic-ppc.cc » ('j') | src/ic/x64/stub-cache-x64.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698