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

Unified Diff: runtime/vm/stub_code_ia32_test.cc

Issue 2992483002: [vm] Update runtime/vm/stub_code_<arch>_test to avoid using Bigints (Closed)
Patch Set: 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
Index: runtime/vm/stub_code_ia32_test.cc
diff --git a/runtime/vm/stub_code_ia32_test.cc b/runtime/vm/stub_code_ia32_test.cc
index b70f19a85dc5d6d165bcf0cf2ccb6b0c3bc0b844..460f521d8d15be4165c57308e99f042bb6fa4c1c 100644
--- a/runtime/vm/stub_code_ia32_test.cc
+++ b/runtime/vm/stub_code_ia32_test.cc
@@ -67,20 +67,25 @@ 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));
__ enter(Immediate(0));
- __ ReserveAlignedFrameSpace(2 * kWordSize);
- __ LoadObject(EAX, bigint1);
- __ movl(Address(ESP, 0), EAX); // Push argument 1 bigint1.
- __ LoadObject(EAX, bigint2);
- __ movl(Address(ESP, kWordSize), EAX); // Push argument 2 bigint2.
- __ CallRuntime(kBigintCompareRuntimeEntry, 2);
- __ SmiTag(EAX);
+ __ ReserveAlignedFrameSpace(4 * kWordSize);
+ __ LoadObject(EAX, str);
+ __ movl(Address(ESP, 0), EAX); // Push argument 1.
+ __ LoadObject(EAX, lhs_index);
+ __ movl(Address(ESP, kWordSize), EAX); // Push argument 2.
+ __ LoadObject(EAX, rhs_index);
+ __ movl(Address(ESP, 2 * kWordSize), EAX); // Push argument 3.
+ __ LoadObject(EAX, length);
+ __ movl(Address(ESP, 3 * kWordSize), EAX); // Push argument 4.
+ __ CallRuntime(kCaseInsensitiveCompareUC16RuntimeEntry, 4);
__ leave();
__ ret(); // Return value is in EAX.
}
@@ -88,17 +93,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);
+ GenerateCallToCallLeafRuntimeStub(&_assembler_, str_value, lhs_index_value,
+ rhs_index_value, length_value);
const Code& code = Code::Handle(Code::FinalizeCode(
*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

Powered by Google App Engine
This is Rietveld 408576698