diff --git a/src/main/java/com/google/devtools/build/lib/bazel/rules/GenericRules.java b/src/main/java/com/google/devtools/build/lib/bazel/rules/GenericRules.java index e4659e9ebfd8aa..2b81f74ff1737a 100644 --- a/src/main/java/com/google/devtools/build/lib/bazel/rules/GenericRules.java +++ b/src/main/java/com/google/devtools/build/lib/bazel/rules/GenericRules.java @@ -46,16 +46,7 @@ public void init(ConfiguredRuleClassProvider.Builder builder) { try { builder.addWorkspaceFilePrefix( - ResourceFileLoader.loadResource(BazelRuleClassProvider.class, "tools.WORKSPACE") - // Hackily select the java_toolchain based on the host JDK version. JDK 8 and - // 9 host_javabases require different toolchains, e.g. to use --patch-module - // instead of -Xbootclasspath/p:. - .replace( - "%java_toolchain%", - isJdk8OrEarlier() - ? "@bazel_tools//tools/jdk:toolchain_hostjdk8" - : "@bazel_tools//tools/jdk:toolchain_hostjdk10")); - + ResourceFileLoader.loadResource(BazelRuleClassProvider.class, "tools.WORKSPACE")); } catch (IOException e) { throw new IllegalStateException(e); } @@ -65,8 +56,4 @@ public void init(ConfiguredRuleClassProvider.Builder builder) { public ImmutableList requires() { return ImmutableList.of(CoreRules.INSTANCE); } - - private static boolean isJdk8OrEarlier() { - return Double.parseDouble(System.getProperty("java.class.version")) <= 52.0; - } } diff --git a/src/main/java/com/google/devtools/build/lib/bazel/rules/tools.WORKSPACE b/src/main/java/com/google/devtools/build/lib/bazel/rules/tools.WORKSPACE index 49f1b5d85a1f3e..b080717b433af8 100644 --- a/src/main/java/com/google/devtools/build/lib/bazel/rules/tools.WORKSPACE +++ b/src/main/java/com/google/devtools/build/lib/bazel/rules/tools.WORKSPACE @@ -1,3 +1,2 @@ local_repository(name = "bazel_tools", path = __embedded_dir__ + "/embedded_tools") bind(name = "cc_toolchain", actual = "@bazel_tools//tools/cpp:default-toolchain") -bind(name = "java_toolchain", actual = "%java_toolchain%") diff --git a/tools/jdk/BUILD b/tools/jdk/BUILD index 689bc138d74c54..454d6742b74d31 100644 --- a/tools/jdk/BUILD +++ b/tools/jdk/BUILD @@ -3,8 +3,6 @@ package(default_visibility = ["//visibility:public"]) load( "//tools/jdk:default_java_toolchain.bzl", "default_java_toolchain", - "JDK8_JVM_OPTS", - "JDK10_JVM_OPTS", "DEFAULT_JAVACOPTS", ) @@ -195,27 +193,15 @@ genrule( ) default_java_toolchain( - name = "toolchain_hostjdk8", - bootclasspath = [":bootclasspath"], - extclasspath = [":extclasspath"], - jvm_opts = JDK8_JVM_OPTS, - source_version = "8", - target_version = "8", -) - -default_java_toolchain( - name = "toolchain_hostjdk10", + name = "toolchain", bootclasspath = [":bootclasspath"], extclasspath = [":extclasspath"], - jvm_opts = JDK10_JVM_OPTS, source_version = "8", target_version = "8", ) default_java_toolchain( name = "toolchain_java9", - # We use the same flags for JDK9 and 10 - jvm_opts = JDK10_JVM_OPTS, misc = DEFAULT_JAVACOPTS + [ "--release", "9", @@ -224,18 +210,12 @@ default_java_toolchain( default_java_toolchain( name = "toolchain_java10", - jvm_opts = JDK10_JVM_OPTS, misc = DEFAULT_JAVACOPTS + [ "--release", "10", ], ) -alias( - name = "toolchain", - actual = "//external:java_toolchain", -) - filegroup( name = "srcs", srcs = [ diff --git a/tools/jdk/default_java_toolchain.bzl b/tools/jdk/default_java_toolchain.bzl index 1cd5619cc8459c..db5e3e8616fda0 100644 --- a/tools/jdk/default_java_toolchain.bzl +++ b/tools/jdk/default_java_toolchain.bzl @@ -14,11 +14,7 @@ """Bazel rules for creating Java toolchains.""" -JDK8_JVM_OPTS = [ - "-Xbootclasspath/p:$(location @bazel_tools//third_party/java/jdk/langtools:javac_jar)", -] - -JDK10_JVM_OPTS = [ +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", @@ -75,7 +71,7 @@ DEFAULT_TOOLCHAIN_CONFIGURATION = { "@bazel_tools//third_party/java/jdk/langtools:jdk_compiler_jar", ], "javac_supports_workers": 1, - "jvm_opts": JDK8_JVM_OPTS, + "jvm_opts": JVM_OPTS, "misc": DEFAULT_JAVACOPTS, "compatible_javacopts": COMPATIBLE_JAVACOPTS, "singlejar": ["@bazel_tools//tools/jdk:singlejar"],