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

Unified Diff: runtime/vm/stub_code_x64_test.cc

Issue 2992483002: [vm] Update runtime/vm/stub_code_<arch>_test to avoid using Bigints (Closed)
Patch Set: Cleanup: _assembler_ locals are renamed to assembler. Created 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « runtime/vm/stub_code_ia32_test.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/stub_code_x64_test.cc
diff --git a/runtime/vm/stub_code_x64_test.cc b/runtime/vm/stub_code_x64_test.cc
index 5a74fbc17330b5074257a2ee80efb8fae1668fc6..0e282f0c0b2361396a46c40b8a3aa640ce06fdfd 100644
--- a/runtime/vm/stub_code_x64_test.cc
+++ b/runtime/vm/stub_code_x64_test.cc
@@ -55,10 +55,10 @@ TEST_CASE(CallRuntimeStubCode) {
const Code& code);
const int length = 10;
const char* kName = "Test_CallRuntimeStubCode";
- Assembler _assembler_;
- GenerateCallToCallRuntimeStub(&_assembler_, length);
+ Assembler assembler;
+ GenerateCallToCallRuntimeStub(&assembler, length);
const Code& code = Code::Handle(Code::FinalizeCode(
- *CreateFunction("Test_CallRuntimeStubCode"), &_assembler_));
+ *CreateFunction("Test_CallRuntimeStubCode"), &assembler));
const Function& function = RegisterFakeFunction(kName, code);
Array& result = Array::Handle();
result ^= DartEntry::InvokeFunction(function, Object::empty_array());
@@ -67,18 +67,21 @@ TEST_CASE(CallRuntimeStubCode) {
// Test calls to stub code which calls into a leaf runtime entry.
static void GenerateCallToCallLeafRuntimeStub(Assembler* assembler,
- const char* value1,
- const char* value2) {
- const Bigint& bigint1 =
- Bigint::ZoneHandle(Bigint::NewFromCString(value1, Heap::kOld));
- const Bigint& bigint2 =
- Bigint::ZoneHandle(Bigint::NewFromCString(value2, Heap::kOld));
+ const char* str_value,
+ intptr_t lhs_index_value,
+ intptr_t rhs_index_value,
+ intptr_t length_value) {
+ const String& str = String::ZoneHandle(String::New(str_value, Heap::kOld));
+ const Smi& lhs_index = Smi::ZoneHandle(Smi::New(lhs_index_value));
+ const Smi& rhs_index = Smi::ZoneHandle(Smi::New(rhs_index_value));
+ const Smi& length = Smi::ZoneHandle(Smi::New(length_value));
__ EnterStubFrame();
__ ReserveAlignedFrameSpace(0);
- __ LoadObject(CallingConventions::kArg1Reg, bigint1);
- __ LoadObject(CallingConventions::kArg2Reg, bigint2);
- __ CallRuntime(kBigintCompareRuntimeEntry, 2);
- __ SmiTag(RAX);
+ __ LoadObject(CallingConventions::kArg1Reg, str);
+ __ LoadObject(CallingConventions::kArg2Reg, lhs_index);
+ __ LoadObject(CallingConventions::kArg3Reg, rhs_index);
+ __ LoadObject(CallingConventions::kArg4Reg, length);
+ __ CallRuntime(kCaseInsensitiveCompareUC16RuntimeEntry, 4);
__ LeaveStubFrame();
__ ret(); // Return value is in RAX.
}
@@ -86,17 +89,20 @@ static void GenerateCallToCallLeafRuntimeStub(Assembler* assembler,
TEST_CASE(CallLeafRuntimeStubCode) {
extern const Function& RegisterFakeFunction(const char* name,
const Code& code);
- const char* value1 = "0xAAABBCCDDAABBCCDD";
- const char* value2 = "0xAABBCCDDAABBCCDD";
+ const char* str_value = "abAB";
+ intptr_t lhs_index_value = 0;
+ intptr_t rhs_index_value = 2;
+ intptr_t length_value = 2;
const char* kName = "Test_CallLeafRuntimeStubCode";
- Assembler _assembler_;
- GenerateCallToCallLeafRuntimeStub(&_assembler_, value1, value2);
+ Assembler assembler;
+ GenerateCallToCallLeafRuntimeStub(&assembler, str_value, lhs_index_value,
+ rhs_index_value, length_value);
const Code& code = Code::Handle(Code::FinalizeCode(
- *CreateFunction("Test_CallLeafRuntimeStubCode"), &_assembler_));
+ *CreateFunction("Test_CallLeafRuntimeStubCode"), &assembler));
const Function& function = RegisterFakeFunction(kName, code);
- Smi& result = Smi::Handle();
+ Instance& result = Instance::Handle();
result ^= DartEntry::InvokeFunction(function, Object::empty_array());
- EXPECT_EQ(1, result.Value());
+ EXPECT_EQ(Bool::True().raw(), result.raw());
}
} // namespace dart
« no previous file with comments | « runtime/vm/stub_code_ia32_test.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698