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

Side by Side Diff: test/unittests/compiler/js-typed-lowering-unittest.cc

Issue 650073002: vector-based ICs did not update type feedback counts correctly. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: REBASE. Created 6 years, 2 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 | Annotate | Revision Log
« no previous file with comments | « test/cctest/test-heap.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 the V8 project authors. All rights reserved. 1 // Copyright 2014 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 #include "src/compiler/access-builder.h" 5 #include "src/compiler/access-builder.h"
6 #include "src/compiler/js-graph.h" 6 #include "src/compiler/js-graph.h"
7 #include "src/compiler/js-operator.h" 7 #include "src/compiler/js-operator.h"
8 #include "src/compiler/js-typed-lowering.h" 8 #include "src/compiler/js-typed-lowering.h"
9 #include "src/compiler/machine-operator.h" 9 #include "src/compiler/machine-operator.h"
10 #include "src/compiler/node-properties-inl.h" 10 #include "src/compiler/node-properties-inl.h"
(...skipping 60 matching lines...) Expand 10 before | Expand all | Expand 10 after
71 // ----------------------------------------------------------------------------- 71 // -----------------------------------------------------------------------------
72 // JSLoadProperty 72 // JSLoadProperty
73 73
74 74
75 TEST_F(JSTypedLoweringTest, JSLoadPropertyFromExternalTypedArray) { 75 TEST_F(JSTypedLoweringTest, JSLoadPropertyFromExternalTypedArray) {
76 const size_t kLength = 17; 76 const size_t kLength = 17;
77 uint8_t backing_store[kLength * 8]; 77 uint8_t backing_store[kLength * 8];
78 Handle<JSArrayBuffer> buffer = 78 Handle<JSArrayBuffer> buffer =
79 NewArrayBuffer(backing_store, arraysize(backing_store)); 79 NewArrayBuffer(backing_store, arraysize(backing_store));
80 VectorSlotPair feedback(Handle<TypeFeedbackVector>::null(), 80 VectorSlotPair feedback(Handle<TypeFeedbackVector>::null(),
81 FeedbackVectorSlot::Invalid()); 81 FeedbackVectorICSlot::Invalid());
82 TRACED_FOREACH(ExternalArrayType, type, kExternalArrayTypes) { 82 TRACED_FOREACH(ExternalArrayType, type, kExternalArrayTypes) {
83 Handle<JSTypedArray> array = 83 Handle<JSTypedArray> array =
84 factory()->NewJSTypedArray(type, buffer, 0, kLength); 84 factory()->NewJSTypedArray(type, buffer, 0, kLength);
85 85
86 Node* key = Parameter(Type::Integral32()); 86 Node* key = Parameter(Type::Integral32());
87 Node* base = HeapConstant(array); 87 Node* base = HeapConstant(array);
88 Node* context = UndefinedConstant(); 88 Node* context = UndefinedConstant();
89 Node* effect = graph()->start(); 89 Node* effect = graph()->start();
90 Node* control = graph()->start(); 90 Node* control = graph()->start();
91 Node* node = graph()->NewNode(javascript()->LoadProperty(feedback), base, 91 Node* node = graph()->NewNode(javascript()->LoadProperty(feedback), base,
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
144 IsIntPtrConstant(bit_cast<intptr_t>(&backing_store[0])), 144 IsIntPtrConstant(bit_cast<intptr_t>(&backing_store[0])),
145 key, IsInt32Constant(static_cast<int>(kLength)), value, 145 key, IsInt32Constant(static_cast<int>(kLength)), value,
146 effect, control)); 146 effect, control));
147 } 147 }
148 } 148 }
149 } 149 }
150 150
151 } // namespace compiler 151 } // namespace compiler
152 } // namespace internal 152 } // namespace internal
153 } // namespace v8 153 } // namespace v8
OLDNEW
« no previous file with comments | « test/cctest/test-heap.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698