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

Side by Side Diff: test/cctest/compiler/test-js-constant-cache.cc

Issue 1314473007: [turbofan] Remove usage of Unique<T> from graph. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Rebased and fixed. Created 5 years, 3 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 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/v8.h" 5 #include "src/v8.h"
6 6
7 #include "src/assembler.h" 7 #include "src/assembler.h"
8 #include "src/compiler/js-graph.h" 8 #include "src/compiler/js-graph.h"
9 #include "src/compiler/node-properties.h" 9 #include "src/compiler/node-properties.h"
10 #include "src/compiler/typer.h" 10 #include "src/compiler/typer.h"
(...skipping 29 matching lines...) Expand all
40 : JSCacheTesterHelper(main_isolate(), main_zone()), 40 : JSCacheTesterHelper(main_isolate(), main_zone()),
41 JSGraph(main_isolate(), &main_graph_, &main_common_, &main_javascript_, 41 JSGraph(main_isolate(), &main_graph_, &main_common_, &main_javascript_,
42 &main_machine_) { 42 &main_machine_) {
43 main_graph_.SetStart(main_graph_.NewNode(common()->Start(0))); 43 main_graph_.SetStart(main_graph_.NewNode(common()->Start(0)));
44 main_graph_.SetEnd(main_graph_.NewNode(common()->End(1))); 44 main_graph_.SetEnd(main_graph_.NewNode(common()->End(1)));
45 main_typer_.Run(); 45 main_typer_.Run();
46 } 46 }
47 47
48 Type* upper(Node* node) { return NodeProperties::GetBounds(node).upper; } 48 Type* upper(Node* node) { return NodeProperties::GetBounds(node).upper; }
49 49
50 Handle<Object> handle(Node* node) { 50 Handle<HeapObject> handle(Node* node) {
51 CHECK_EQ(IrOpcode::kHeapConstant, node->opcode()); 51 CHECK_EQ(IrOpcode::kHeapConstant, node->opcode());
52 return OpParameter<Unique<Object> >(node).handle(); 52 return OpParameter<Handle<HeapObject>>(node);
53 } 53 }
54 54
55 Factory* factory() { return main_isolate()->factory(); } 55 Factory* factory() { return main_isolate()->factory(); }
56 }; 56 };
57 57
58 58
59 TEST(ZeroConstant1) { 59 TEST(ZeroConstant1) {
60 JSConstantCacheTester T; 60 JSConstantCacheTester T;
61 61
62 Node* zero = T.ZeroConstant(); 62 Node* zero = T.ZeroConstant();
(...skipping 402 matching lines...) Expand 10 before | Expand all | Expand 10 after
465 T.Constant(1.11), 465 T.Constant(1.11),
466 T.ExternalConstant(ExternalReference::address_of_one_half())}; 466 T.ExternalConstant(ExternalReference::address_of_one_half())};
467 467
468 NodeVector nodes(T.main_zone()); 468 NodeVector nodes(T.main_zone());
469 T.GetCachedNodes(&nodes); 469 T.GetCachedNodes(&nodes);
470 470
471 for (size_t i = 0; i < arraysize(constants); i++) { 471 for (size_t i = 0; i < arraysize(constants); i++) {
472 CHECK(Contains(&nodes, constants[i])); 472 CHECK(Contains(&nodes, constants[i]));
473 } 473 }
474 } 474 }
OLDNEW
« no previous file with comments | « test/cctest/compiler/graph-builder-tester.h ('k') | test/cctest/compiler/test-js-context-specialization.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698