Avoid making a copy in mergeMaps if one of the input maps is empty.

PiperOrigin-RevId: 615354375 Change-Id: I6914444b5e4cd30e8519a9df41eb546d3e9aaa81
bazelbuild · Mar 14, 2024 · 3e8ee27 · 3e8ee27
1 parent 2288c9c
commit 3e8ee27
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/src/main/java/com/google/devtools/build/lib/actions/ActionAnalysisMetadata.java b/src/main/java/com/google/devtools/build/lib/actions/ActionAnalysisMetadata.java
@@ -253,6 +253,12 @@ default ImmutableMap<String, String> getExecutionInfo() {
 
   static ImmutableMap<String, String> mergeMaps(
       ImmutableMap<String, String> first, ImmutableMap<String, String> second) {
+    if (first.isEmpty()) {
+      return second;
+    }
+    if (second.isEmpty()) {
+      return first;
+    }
     return ImmutableMap.<String, String>builderWithExpectedSize(first.size() + second.size())
         .putAll(first)
         .putAll(second)