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

Side by Side Diff: runtime/vm/stub_code_arm_test.cc

Issue 23672011: Fix the previously ineffective assert checking the number of arguments passed to (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 7 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 | Annotate | Revision Log
« no previous file with comments | « runtime/vm/stub_code_arm.cc ('k') | runtime/vm/stub_code_ia32.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 #include "vm/globals.h" 5 #include "vm/globals.h"
6 #if defined(TARGET_ARCH_ARM) 6 #if defined(TARGET_ARCH_ARM)
7 7
8 #include "vm/isolate.h" 8 #include "vm/isolate.h"
9 #include "vm/dart_entry.h" 9 #include "vm/dart_entry.h"
10 #include "vm/native_entry.h" 10 #include "vm/native_entry.h"
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
44 const Smi& smi2 = Smi::ZoneHandle(Smi::New(value2)); 44 const Smi& smi2 = Smi::ZoneHandle(Smi::New(value2));
45 const Object& result = Object::ZoneHandle(); 45 const Object& result = Object::ZoneHandle();
46 const Context& context = Context::ZoneHandle(Context::New(0, Heap::kOld)); 46 const Context& context = Context::ZoneHandle(Context::New(0, Heap::kOld));
47 ASSERT(context.isolate() == Isolate::Current()); 47 ASSERT(context.isolate() == Isolate::Current());
48 __ EnterDartFrame(0); 48 __ EnterDartFrame(0);
49 __ LoadObject(CTX, context); 49 __ LoadObject(CTX, context);
50 __ PushObject(result); // Push Null object for return value. 50 __ PushObject(result); // Push Null object for return value.
51 __ PushObject(smi1); // Push argument 1 smi1. 51 __ PushObject(smi1); // Push argument 1 smi1.
52 __ PushObject(smi2); // Push argument 2 smi2. 52 __ PushObject(smi2); // Push argument 2 smi2.
53 ASSERT(kTestSmiSubRuntimeEntry.argument_count() == argc); 53 ASSERT(kTestSmiSubRuntimeEntry.argument_count() == argc);
54 __ CallRuntime(kTestSmiSubRuntimeEntry); // Call SmiSub runtime func. 54 __ CallRuntime(kTestSmiSubRuntimeEntry, argc); // Call SmiSub runtime func.
55 __ AddImmediate(SP, argc * kWordSize); 55 __ AddImmediate(SP, argc * kWordSize);
56 __ Pop(R0); // Pop return value from return slot. 56 __ Pop(R0); // Pop return value from return slot.
57 __ LeaveDartFrame(); 57 __ LeaveDartFrame();
58 __ Ret(); 58 __ Ret();
59 } 59 }
60 60
61 61
62 TEST_CASE(CallRuntimeStubCode) { 62 TEST_CASE(CallRuntimeStubCode) {
63 extern const Function& RegisterFakeFunction(const char* name, 63 extern const Function& RegisterFakeFunction(const char* name,
64 const Code& code); 64 const Code& code);
(...skipping 14 matching lines...) Expand all
79 // Test calls to stub code which calls into a leaf runtime entry. 79 // Test calls to stub code which calls into a leaf runtime entry.
80 static void GenerateCallToCallLeafRuntimeStub(Assembler* assembler, 80 static void GenerateCallToCallLeafRuntimeStub(Assembler* assembler,
81 int value1, 81 int value1,
82 int value2) { 82 int value2) {
83 const Smi& smi1 = Smi::ZoneHandle(Smi::New(value1)); 83 const Smi& smi1 = Smi::ZoneHandle(Smi::New(value1));
84 const Smi& smi2 = Smi::ZoneHandle(Smi::New(value2)); 84 const Smi& smi2 = Smi::ZoneHandle(Smi::New(value2));
85 __ EnterDartFrame(0); 85 __ EnterDartFrame(0);
86 __ ReserveAlignedFrameSpace(0); 86 __ ReserveAlignedFrameSpace(0);
87 __ LoadObject(R0, smi1); // Set up argument 1 smi1. 87 __ LoadObject(R0, smi1); // Set up argument 1 smi1.
88 __ LoadObject(R1, smi2); // Set up argument 2 smi2. 88 __ LoadObject(R1, smi2); // Set up argument 2 smi2.
89 __ CallRuntime(kTestLeafSmiAddRuntimeEntry); // Call SmiAdd runtime func. 89 __ CallRuntime(kTestLeafSmiAddRuntimeEntry, 2); // Call SmiAdd runtime func.
90 __ LeaveDartFrame(); 90 __ LeaveDartFrame();
91 __ Ret(); // Return value is in R0. 91 __ Ret(); // Return value is in R0.
92 } 92 }
93 93
94 94
95 TEST_CASE(CallLeafRuntimeStubCode) { 95 TEST_CASE(CallLeafRuntimeStubCode) {
96 extern const Function& RegisterFakeFunction(const char* name, 96 extern const Function& RegisterFakeFunction(const char* name,
97 const Code& code); 97 const Code& code);
98 const int value1 = 10; 98 const int value1 = 10;
99 const int value2 = 20; 99 const int value2 = 20;
100 const char* kName = "Test_CallLeafRuntimeStubCode"; 100 const char* kName = "Test_CallLeafRuntimeStubCode";
101 Assembler _assembler_; 101 Assembler _assembler_;
102 GenerateCallToCallLeafRuntimeStub(&_assembler_, value1, value2); 102 GenerateCallToCallLeafRuntimeStub(&_assembler_, value1, value2);
103 const Code& code = Code::Handle(Code::FinalizeCode( 103 const Code& code = Code::Handle(Code::FinalizeCode(
104 *CreateFunction("Test_CallLeafRuntimeStubCode"), &_assembler_)); 104 *CreateFunction("Test_CallLeafRuntimeStubCode"), &_assembler_));
105 const Function& function = RegisterFakeFunction(kName, code); 105 const Function& function = RegisterFakeFunction(kName, code);
106 Smi& result = Smi::Handle(); 106 Smi& result = Smi::Handle();
107 result ^= DartEntry::InvokeFunction(function, Object::empty_array()); 107 result ^= DartEntry::InvokeFunction(function, Object::empty_array());
108 EXPECT_EQ((value1 + value2), result.Value()); 108 EXPECT_EQ((value1 + value2), result.Value());
109 } 109 }
110 110
111 } // namespace dart 111 } // namespace dart
112 112
113 #endif // defined TARGET_ARCH_ARM 113 #endif // defined TARGET_ARCH_ARM
OLDNEW
« no previous file with comments | « runtime/vm/stub_code_arm.cc ('k') | runtime/vm/stub_code_ia32.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698