Hi Igor, On 12/04/2017 12:29 PM, Igor Ignatyev wrote:

http://cr.openjdk.java.net/~iignatyev/8178415/webrev.00/

...

59 lines changed: 0 ins; 59 del; 0 mod;

Hi all,

could you please review this small clean up of testlibaries? the only platform specific vm arguments which we had were -d64/32. they were deprecated in jdk9 by 8168010 and will be removed in jdk10 by 8169646. therefore ProcessTools::getPlatformSpecificVMArgs method is not needed anymore and should be removed.

JBS: https://bugs.openjdk.java.net/browse/JDK-8178415 webrev: http://cr.openjdk.java.net/~iignatyev/8178415/webrev.00/ testing: :jdk_tier{1-3}, :hotspot_all 8168010: https://bugs.openjdk.java.net/browse/JDK-8168010 8169646: https://bugs.openjdk.java.net/browse/JDK-8169646

These changes look good to me. Thanks, David

Thanks, -- Igor

Hi Igor, The fix looks, thanks for doing it. Please update the copyrights before integration :) Thanks, Dima On 12.04.2017 5:29, Igor Ignatyev wrote:

http://cr.openjdk.java.net/~iignatyev/8178415/webrev.00/

...

59 lines changed: 0 ins; 59 del; 0 mod; Hi all,

could you please review this small clean up of testlibaries? the only platform specific vm arguments which we had were -d64/32. they were deprecated in jdk9 by 8168010 and will be removed in jdk10 by 8169646. therefore ProcessTools::getPlatformSpecificVMArgs method is not needed anymore and should be removed.

JBS: https://bugs.openjdk.java.net/browse/JDK-8178415 webrev: http://cr.openjdk.java.net/~iignatyev/8178415/webrev.00/ testing: :jdk_tier{1-3}, :hotspot_all 8168010: https://bugs.openjdk.java.net/browse/JDK-8168010 8169646: https://bugs.openjdk.java.net/browse/JDK-8169646

Thanks, -- Igor