From a5ca8ebef24e38348213cef31417a4ccc7c3dd1a Mon Sep 17 00:00:00 2001
From: Nicolas Geoffray <ngeoffray@google.com>
Date: Fri, 24 Aug 2018 13:39:13 +0100
Subject: Default to usejit:true.

Manually verified places that could be affected (go/lem, buildbots, CTS).
Also adjust two tests expecting JIT.

bug: 64164063
Test: m and tests
Change-Id: I6dc441f0f58203546480a0f7220ad4d9cd593fd3
---
 test/638-checker-inline-cache-intrinsic/run |  2 ++
 test/916-obsolete-jit/run                   | 10 +---------
 test/etc/run-test-jar                       |  6 ++++--
 test/knownfailures.json                     |  7 ++++++-
 4 files changed, 13 insertions(+), 12 deletions(-)

(limited to 'test')

diff --git a/test/638-checker-inline-cache-intrinsic/run b/test/638-checker-inline-cache-intrinsic/run
index f43681dd56..15403100f5 100644
--- a/test/638-checker-inline-cache-intrinsic/run
+++ b/test/638-checker-inline-cache-intrinsic/run
@@ -14,4 +14,6 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+# Set threshold to 100 to math the iterations done in the test.
+# Pass --verbose-methods to only generate the CFG of these methods.
 exec ${RUN} --jit --runtime-option -Xjitthreshold:100 -Xcompiler-option --verbose-methods=inlineMonomorphic,knownReceiverType,stringEquals $@
diff --git a/test/916-obsolete-jit/run b/test/916-obsolete-jit/run
index b6d406fd99..c6e62ae6cd 100755
--- a/test/916-obsolete-jit/run
+++ b/test/916-obsolete-jit/run
@@ -14,12 +14,4 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-# We are testing the redefinition of compiled code but with jvmti we only allow
-# jitted compiled code so always add the --jit argument.
-if [[ "$@" == *"--jit"* ]]; then
-  other_args=""
-else
-  other_args="--jit"
-fi
-./default-run "$@" ${other_args} \
-                   --jvmti
+./default-run "$@" --jvmti
diff --git a/test/etc/run-test-jar b/test/etc/run-test-jar
index 713fd35523..5c4ad21bd6 100755
--- a/test/etc/run-test-jar
+++ b/test/etc/run-test-jar
@@ -588,7 +588,7 @@ if [ "$USE_GDB" = "y" ]; then
 fi
 
 if [ "$INTERPRETER" = "y" ]; then
-    INT_OPTS="-Xint"
+    INT_OPTS="${INT_OPTS} -Xint"
     if [ "$VERIFY" = "y" ] ; then
       INT_OPTS="${INT_OPTS} -Xcompiler-option --compiler-filter=quicken"
       COMPILE_FLAGS="${COMPILE_FLAGS} --compiler-filter=quicken"
@@ -604,7 +604,7 @@ if [ "$INTERPRETER" = "y" ]; then
 fi
 
 if [ "$JIT" = "y" ]; then
-    INT_OPTS="-Xusejit:true"
+    INT_OPTS="${INT_OPTS} -Xusejit:true"
     if [ "$VERIFY" = "y" ] ; then
       INT_OPTS="${INT_OPTS} -Xcompiler-option --compiler-filter=quicken"
       COMPILE_FLAGS="${COMPILE_FLAGS} --compiler-filter=quicken"
@@ -613,6 +613,8 @@ if [ "$JIT" = "y" ]; then
       COMPILE_FLAGS="${COMPILE_FLAGS} --compiler-filter=assume-verified"
       DEX_VERIFY="${DEX_VERIFY} -Xverify:none"
     fi
+else
+    INT_OPTS="${INT_OPTS} -Xusejit:false"
 fi
 
 JNI_OPTS="-Xjnigreflimit:512 -Xcheck:jni"
diff --git a/test/knownfailures.json b/test/knownfailures.json
index a280f8522b..f71e9363cb 100644
--- a/test/knownfailures.json
+++ b/test/knownfailures.json
@@ -246,7 +246,7 @@
                   "613-inlining-dex-cache",
                   "626-set-resolved-string",
                   "638-checker-inline-cache-intrinsic"],
-        "variant": "trace  | stream",
+        "variant": "trace | stream",
         "description": ["These tests expect JIT compilation, which is",
                         "suppressed when tracing."]
     },
@@ -256,6 +256,11 @@
         "description": ["This test expects JIT compilation, which is",
                         "suppressed when tracing."]
     },
+    {
+        "tests": "638-checker-inline-cache-intrinsic",
+        "variant": "interpreter | interp-ac",
+        "description": ["Test expects JIT compilation"]
+    },
     {
         "tests": "597-deopt-invoke-stub",
         "variant": "speed-profile | interp-ac | interpreter | optimizing | trace | stream",
-- 
cgit v1.2.3-59-g8ed1b