From 3e6232ec4574feeef11f61e6a08ce7bc1db8c08f Mon Sep 17 00:00:00 2001 From: Roland Levillain Date: Thu, 11 Feb 2016 12:50:41 +0000 Subject: [PATCH] Make run test 572-checker-array-get-regression gcstress friendly. The large array allocated in test/572-checker-array-get-regression/src/Main.java used to be too big for the small heap used in gcstress mode. Use a smaller array size that however still exercises the initial issue checked by this regression test. Bug: 26817006 Bug: 12687968 Change-Id: Id7080a18007cad9b5d4ac31b45f7df46213a908e --- test/572-checker-array-get-regression/expected.txt | 2 +- test/572-checker-array-get-regression/info.txt | 2 +- test/572-checker-array-get-regression/src/Main.java | 15 ++++++++------- 3 files changed, 10 insertions(+), 9 deletions(-) diff --git a/test/572-checker-array-get-regression/expected.txt b/test/572-checker-array-get-regression/expected.txt index c08d7831c..f7d1ad474 100644 --- a/test/572-checker-array-get-regression/expected.txt +++ b/test/572-checker-array-get-regression/expected.txt @@ -1 +1 @@ -1048575 +524287 diff --git a/test/572-checker-array-get-regression/info.txt b/test/572-checker-array-get-regression/info.txt index 023d5b0c1..d06feee15 100644 --- a/test/572-checker-array-get-regression/info.txt +++ b/test/572-checker-array-get-regression/info.txt @@ -1,3 +1,3 @@ Regression test for the ARM64 Baker's read barrier fast path compiler -instrumentatin of array loads with a large constant index, where we +instrumentation of array loads with a large constant index, where we used to require too many scratch (temporary) registers. diff --git a/test/572-checker-array-get-regression/src/Main.java b/test/572-checker-array-get-regression/src/Main.java index 139221555..a9bf32642 100644 --- a/test/572-checker-array-get-regression/src/Main.java +++ b/test/572-checker-array-get-regression/src/Main.java @@ -22,9 +22,9 @@ public class Main { /// CHECK-START: java.lang.Integer Main.test() ssa_builder (after) /// CHECK-DAG: <> CurrentMethod - /// CHECK-DAG: <> IntConstant 1048576 + /// CHECK-DAG: <> IntConstant 524288 /// CHECK-DAG: <> IntConstant -1 - /// CHECK-DAG: <> NewArray [<>,<>] + /// CHECK-DAG: <> NewArray [<>,<>] /// CHECK-DAG: <> NullCheck [<>] /// CHECK-DAG: <> ArrayLength [<>] /// CHECK-DAG: <> Add [<>,<>] @@ -34,16 +34,17 @@ public class Main { /// CHECK-DAG: <> ArrayGet [<>,<>] /// CHECK-DAG: Return [<>] + /// CHECK-START: java.lang.Integer Main.test() register (before) /// CHECK-DAG: <> CurrentMethod - /// CHECK-DAG: <> IntConstant 1048576 - /// CHECK-DAG: <> IntConstant 1048575 - /// CHECK-DAG: <> NewArray [<>,<>] - /// CHECK-DAG: <> ArrayGet [<>,<>] + /// CHECK-DAG: <> IntConstant 524288 + /// CHECK-DAG: <> IntConstant 524287 + /// CHECK-DAG: <> NewArray [<>,<>] + /// CHECK-DAG: <> ArrayGet [<>,<>] /// CHECK-DAG: Return [<>] public static Integer test() { - Integer[] integers = new Integer[1024 * 1024]; + Integer[] integers = new Integer[1 << 19]; initIntegerArray(integers); // Array load with a large constant index (after constant folding // and bounds check elimination). -- 2.11.0