commit ec4c7752c22ae26924378d953a6f4a0e8e8b036d
author Brian Carlstrom <bdc@google.com> Mon Mar 10 10:47:26 2014 -0700
committer Android Git Automerger <android-git-automerger@android.com> Mon Mar 10 10:47:26 2014 -0700
tree 5fafe2a2af344fa6da3098b7f973103e9514f03f
parent bbf826659e8e5d0be683b2459207c495303d53b6
parent ab97d617e8043f7396dc6031c4ddeb2ae632230f

am ab97d617: Merge "Only pass -Xprofile-* options to ART"

* commit 'ab97d617e8043f7396dc6031c4ddeb2ae632230f':
  Only pass -Xprofile-* options to ART

core/jni/AndroidRuntime.cpp

diff --git a/core/jni/AndroidRuntime.cpp b/core/jni/AndroidRuntime.cpp
index e623d5d..98f7716 100644
--- a/core/jni/AndroidRuntime.cpp
+++ b/core/jni/AndroidRuntime.cpp
@@ -749,10 +749,11 @@
         mOptions.add(opt);
     }
 
-    // libart tolerates libdvm flags, but not vice versa, so only pass these if libart.
+    // libart tolerates libdvm flags, but not vice versa, so only pass some options if libart.
     property_get("persist.sys.dalvik.vm.lib.1", dalvikVmLibBuf, "libdvm.so");
-    if (strncmp(dalvikVmLibBuf, "libart", 6) == 0) {
+    bool libart = (strncmp(dalvikVmLibBuf, "libart", 6) == 0);
 
+    if (libart) {
         // Extra options for DexClassLoader.
         property_get("dalvik.vm.dex2oat-flags", dex2oatFlagsBuf, "");
         parseExtraOpts(dex2oatFlagsBuf, "-Xcompiler-option");
@@ -783,7 +784,7 @@
     /*
      * Set profiler options
      */
-    {
+    if (libart) {
       char period[sizeof("-Xprofile-period:") + PROPERTY_VALUE_MAX];
       char duration[sizeof("-Xprofile-duration:") + PROPERTY_VALUE_MAX];
       char interval[sizeof("-Xprofile-interval:") + PROPERTY_VALUE_MAX];