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

Unified Diff: runtime/vm/stub_code_arm_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_arm64_test.cc ('k') | runtime/vm/stub_code_ia32_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/stub_code_arm_test.cc
diff --git a/runtime/vm/stub_code_arm_test.cc b/runtime/vm/stub_code_arm_test.cc
index f3154b5273e2a5161eb0db56e8eeb667b1fc0e3a..f4ba65c1f54540a8d1063e05cec687d86b36d357 100644
--- a/runtime/vm/stub_code_arm_test.cc
+++ b/runtime/vm/stub_code_arm_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));
__ EnterDartFrame(0);
__ ReserveAlignedFrameSpace(0);
- __ LoadObject(R0, bigint1); // Set up argument 1 bigint1.
- __ LoadObject(R1, bigint2); // Set up argument 2 bigint2.
- __ CallRuntime(kBigintCompareRuntimeEntry, 2);
- __ SmiTag(R0);
+ __ LoadObject(R0, str);
+ __ LoadObject(R1, lhs_index);
+ __ LoadObject(R2, rhs_index);
+ __ LoadObject(R3, length);
+ __ CallRuntime(kCaseInsensitiveCompareUC16RuntimeEntry, 4);
__ LeaveDartFrame();
__ Ret(); // Return value is in R0.
}
@@ -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_arm64_test.cc ('k') | runtime/vm/stub_code_ia32_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698