summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Jeff Hao <jeffhao@google.com> 2015-02-17 18:01:00 -0800
committer Jeff Hao <jeffhao@google.com> 2015-02-17 18:32:15 -0800
commit6d25419fa2dedbfcfea8b0a3066b064906c23b8a (patch)
tree01c801a23f700f6cc430a50500f3cd3b2d77a373
parent6e27f82193a8f54cd8ecdc8fb2c4c1adadafbaf4 (diff)
Default to 64-bit for tests of methods with varying ISAs for valgrind.
The size of the created methods will technically not be correct, but they will be larger than necessary when the target is changed to 32-bit, so valgrind will not complain. Bug: 19368740 Change-Id: Ic78867b4700248ec6d70f5d7f6d87ce9447ac448
-rw-r--r--runtime/arch/arch_test.cc6
-rw-r--r--runtime/entrypoints/quick/quick_trampoline_entrypoints_test.cc6
2 files changed, 12 insertions, 0 deletions
diff --git a/runtime/arch/arch_test.cc b/runtime/arch/arch_test.cc
index ab6b00bddf..5733ab6361 100644
--- a/runtime/arch/arch_test.cc
+++ b/runtime/arch/arch_test.cc
@@ -24,6 +24,12 @@ namespace art {
class ArchTest : public CommonRuntimeTest {
protected:
+ void SetUpRuntimeOptions(RuntimeOptions *options) OVERRIDE {
+ // Use 64-bit ISA for runtime setup to make method size potentially larger
+ // than necessary (rather than smaller) during CreateCalleeSaveMethod
+ options->push_back(std::make_pair("imageinstructionset", "x86_64"));
+ }
+
static void CheckFrameSize(InstructionSet isa, Runtime::CalleeSaveType type, uint32_t save_size)
NO_THREAD_SAFETY_ANALYSIS {
Runtime* r = Runtime::Current();
diff --git a/runtime/entrypoints/quick/quick_trampoline_entrypoints_test.cc b/runtime/entrypoints/quick/quick_trampoline_entrypoints_test.cc
index 85a0b99ab1..01c17acdcc 100644
--- a/runtime/entrypoints/quick/quick_trampoline_entrypoints_test.cc
+++ b/runtime/entrypoints/quick/quick_trampoline_entrypoints_test.cc
@@ -25,6 +25,12 @@ namespace art {
class QuickTrampolineEntrypointsTest : public CommonRuntimeTest {
protected:
+ void SetUpRuntimeOptions(RuntimeOptions *options) OVERRIDE {
+ // Use 64-bit ISA for runtime setup to make method size potentially larger
+ // than necessary (rather than smaller) during CreateCalleeSaveMethod
+ options->push_back(std::make_pair("imageinstructionset", "x86_64"));
+ }
+
static mirror::ArtMethod* CreateCalleeSaveMethod(InstructionSet isa,
Runtime::CalleeSaveType type)
NO_THREAD_SAFETY_ANALYSIS {