Intern javac options in JavaCompilationInfoProvider

This is almost always the same set of options used for compilation, so we use the same interner.

Of course, this change means, all else remaining the same, we will incur an even larger regression when we switch to pure Starlark instances of `JavaInfo`[^1].

PiperOrigin-RevId: 551185053
Change-Id: I0a528b76625ad27adde675a112c232a99d9e7abe

src/main/java/com/google/devtools/build/lib/rules/java/JavaCompilationHelper.java

@@ -134,6 +134,14 @@ public JavaCompilationHelper(
         additionalInputsForDatabinding);
   }
 
+  @Nullable
+  static ImmutableList<String> internJavacOpts(@Nullable ImmutableList<String> javacOpts) {
+    if (javacOpts == null) {
+      return null;
+    }
+    return javacOptsInterner.intern(javacOpts);
+  }
+
   public void enableJspecify(boolean enableJspecify) {
     this.enableJspecify = enableJspecify;
   }

src/main/java/com/google/devtools/build/lib/rules/java/JavaCompilationInfoProvider.java

@@ -143,7 +143,10 @@ public Builder setBootClasspath(BootClassPathInfo bootClasspath) {
 
     public JavaCompilationInfoProvider build() {
       return new AutoValue_JavaCompilationInfoProvider(
-          javacOpts, runtimeClasspath, compilationClasspath, bootClasspath.bootclasspath());
+          JavaCompilationHelper.internJavacOpts(javacOpts),
+          runtimeClasspath,
+          compilationClasspath,
+          bootClasspath.bootclasspath());
     }
   }