From 872c11ae8d1fe9b74aaaf8874201ab7e0f2906df Mon Sep 17 00:00:00 2001 From: Benjamin Peterson Date: Wed, 24 Jul 2019 01:22:53 -0700 Subject: [PATCH] Allow G1 in tools/jdk/BUILD.java_tools. This is a followup to 01e6e05a35633df92cafc20b28eac15cd1aefa09. (I think BUILD.java_tools was introduced in between the creation of the PR for 01e6e05a35633df92cafc20b28eac15cd1aefa09 and its landing.) Closes #8964. PiperOrigin-RevId: 259695109 --- tools/jdk/BUILD.java_tools | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/tools/jdk/BUILD.java_tools b/tools/jdk/BUILD.java_tools index 711abf6705dd02..64bfc42b240782 100644 --- a/tools/jdk/BUILD.java_tools +++ b/tools/jdk/BUILD.java_tools @@ -27,10 +27,6 @@ java_toolchain( javac = [":javac_jar"], javac_supports_workers = 1, jvm_opts = [ - # In JDK9 we have seen a ~30% slow down in JavaBuilder performance when using - # G1 collector and having compact strings enabled. - "-XX:+UseParallelOldGC", - "-XX:-CompactStrings", # Allow JavaBuilder to access internal javac APIs. "--add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED", "--add-exports=jdk.compiler/com.sun.tools.javac.code=ALL-UNNAMED", @@ -88,10 +84,6 @@ java_toolchain( javac = [":javac_jar"], javac_supports_workers = 1, jvm_opts = [ - # In JDK9 we have seen a ~30% slow down in JavaBuilder performance - # when using G1 collector and having compact strings enabled. - "-XX:+UseParallelOldGC", - "-XX:-CompactStrings", # Allow JavaBuilder to access internal javac APIs. "--add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED", "--add-exports=jdk.compiler/com.sun.tools.javac.code=ALL-UNNAMED", @@ -153,10 +145,6 @@ java_toolchain( javac = [":javac_jar"], javac_supports_workers = 1, jvm_opts = [ - # In JDK9 we have seen a ~30% slow down in JavaBuilder performance when using - # G1 collector and having compact strings enabled. - "-XX:+UseParallelOldGC", - "-XX:-CompactStrings", # Allow JavaBuilder to access internal javac APIs. "--add-exports=jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED", "--add-exports=jdk.compiler/com.sun.tools.javac.code=ALL-UNNAMED",