8178415: remove ProcessTools::getPlatformSpecificVMArgs from testlibary
authoriignatyev
Tue, 11 Apr 2017 21:59:26 -0700
changeset 44595 8177065bcb6f
parent 44593 4c9658735110
child 44596 93455b97e669
8178415: remove ProcessTools::getPlatformSpecificVMArgs from testlibary Reviewed-by: dholmes
jdk/test/lib/testlibrary/jdk/testlibrary/JDKToolLauncher.java
jdk/test/lib/testlibrary/jdk/testlibrary/ProcessTools.java
--- a/jdk/test/lib/testlibrary/jdk/testlibrary/JDKToolLauncher.java	Tue Apr 11 13:55:53 2017 -0700
+++ b/jdk/test/lib/testlibrary/jdk/testlibrary/JDKToolLauncher.java	Tue Apr 11 21:59:26 2017 -0700
@@ -24,7 +24,6 @@
 package jdk.testlibrary;
 
 import java.util.ArrayList;
-import java.util.Arrays;
 import java.util.List;
 
 /**
@@ -60,7 +59,6 @@
         } else {
             executable = JDKToolFinder.getTestJDKTool(tool);
         }
-        vmArgs.addAll(Arrays.asList(ProcessTools.getPlatformSpecificVMArgs()));
     }
 
     /**
--- a/jdk/test/lib/testlibrary/jdk/testlibrary/ProcessTools.java	Tue Apr 11 13:55:53 2017 -0700
+++ b/jdk/test/lib/testlibrary/jdk/testlibrary/ProcessTools.java	Tue Apr 11 21:59:26 2017 -0700
@@ -260,23 +260,6 @@
     }
 
     /**
-     * Get platform specific VM arguments (e.g. -d64 on 64bit Solaris)
-     *
-     * @return String[] with platform specific arguments, empty if there are
-     *         none
-     */
-    public static String[] getPlatformSpecificVMArgs() {
-        String osName = System.getProperty("os.name");
-        String dataModel = System.getProperty("sun.arch.data.model");
-
-        if (osName.equals("SunOS") && dataModel.equals("64")) {
-            return new String[] { "-d64" };
-        }
-
-        return new String[] {};
-    }
-
-    /**
      * Create ProcessBuilder using the java launcher from the jdk to be tested,
      * and with any platform specific arguments prepended.
      *
@@ -301,7 +284,6 @@
 
         ArrayList<String> args = new ArrayList<>();
         args.add(javapath);
-        Collections.addAll(args, getPlatformSpecificVMArgs());
 
         if (addTestVmAndJavaOptions) {
             // -cp is needed to make sure the same classpath is used whether the test is