From 135016a1feee5576014f377e8006057837ee73a2 Mon Sep 17 00:00:00 2001 From: Dmitry Petrochenko Date: Thu, 3 Apr 2014 14:35:54 +0700 Subject: art: Use SIRT::GetAlignedSirtSizeTarget in calling convention Calculate frame size based on SIRT::GetAlignedSirtSizeTarget existing method. Make offset functions pointer-size-dependent for cross-compiling. Add a test to check whether our computations are correct. Change-Id: Ic66daf6f9908890eda906bdcbbc4444c4fef614f Signed-off-by: Dmitry Petrochenko --- runtime/stack_indirect_reference_table_test.cc | 58 ++++++++++++++++++++++++++ 1 file changed, 58 insertions(+) create mode 100644 runtime/stack_indirect_reference_table_test.cc (limited to 'runtime/stack_indirect_reference_table_test.cc') diff --git a/runtime/stack_indirect_reference_table_test.cc b/runtime/stack_indirect_reference_table_test.cc new file mode 100644 index 0000000..72ef6b6 --- /dev/null +++ b/runtime/stack_indirect_reference_table_test.cc @@ -0,0 +1,58 @@ +/* + * Copyright (C) 2014 The Android Open Source Project + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +#include "stack_indirect_reference_table.h" +#include "gtest/gtest.h" + +namespace art { + +// Test the offsets computed for members of StackIndirectReferenceTable. Because of cross-compiling +// it is impossible the use OFFSETOF_MEMBER, so we do some reasonable computations ourselves. This +// test checks whether we do the right thing. +TEST(StackIndirectReferenceTableTest, Offsets) { + // As the members of StackIndirectReferenceTable are private, we cannot use OFFSETOF_MEMBER + // here. So do the inverse: set some data, and access it through pointers created from the offsets. + + StackIndirectReferenceTable test_table(reinterpret_cast(0x1234)); + test_table.SetLink(reinterpret_cast(0x5678)); + test_table.SetNumberOfReferences(0x9ABC); + + byte* table_base_ptr = reinterpret_cast(&test_table); + + { + uintptr_t* link_ptr = reinterpret_cast(table_base_ptr + + StackIndirectReferenceTable::LinkOffset(kPointerSize)); + EXPECT_EQ(*link_ptr, static_cast(0x5678)); + } + + { + uint32_t* num_ptr = reinterpret_cast(table_base_ptr + + StackIndirectReferenceTable::NumberOfReferencesOffset(kPointerSize)); + EXPECT_EQ(*num_ptr, static_cast(0x9ABC)); + } + + { + // Assume sizeof(StackReference) == sizeof(uint32_t) + // TODO: How can we make this assumption-less but still access directly and fully? + EXPECT_EQ(sizeof(StackReference), sizeof(uint32_t)); + + uint32_t* ref_ptr = reinterpret_cast(table_base_ptr + + StackIndirectReferenceTable::ReferencesOffset(kPointerSize)); + EXPECT_EQ(*ref_ptr, static_cast(0x1234)); + } +} + +} // namespace art -- cgit v1.1