Remove Target from TargetPatternPhaseValue. Use the sets of targets that would have been in TargetPatternPhaseValue to construct Postables that would use information from the TargetPatternPhaseValue.

Tag TargetPhasePatternValue with @AutoCodec.

PiperOrigin-RevId: 190958515

1 files changed, 21 insertions, 12 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/LegacyLoadingPhaseRunner.java b/src/main/java/com/google/devtools/build/lib/skyframe/LegacyLoadingPhaseRunner.java
index 3f6d11fbb1..00aa64e5e3 100644
--- a/src/main/java/com/google/devtools/build/lib/skyframe/LegacyLoadingPhaseRunner.java
+++ b/src/main/java/com/google/devtools/build/lib/skyframe/LegacyLoadingPhaseRunner.java
@@ -194,37 +194,46 @@ public final class LegacyLoadingPhaseRunner extends LoadingPhaseRunner {
         ImmutableSet.copyOf(Sets.difference(targetsToLoad, expandedTargetsToLoad));
     long testSuiteTime = timer.stop().elapsed(TimeUnit.MILLISECONDS);
 
+    ImmutableSet<Label> targetLabels =
+        expandedTargetsToLoad.stream().map(Target::getLabel).collect(ImmutableSet.toImmutableSet());
+    ImmutableSet<Label> testsToRunLabels = null;
+    if (testsToRun != null) {
+      testsToRunLabels =
+          testsToRun.stream().map(Target::getLabel).collect(ImmutableSet.toImmutableSet());
+    }
+    ImmutableSet<Label> removedTargetLabels =
+        removedTargets.stream().map(Target::getLabel).collect(ImmutableSet.toImmutableSet());
+
     TargetPatternPhaseValue patternParsingValue =
         new TargetPatternPhaseValue(
-            expandedTargetsToLoad,
-            testsToRun,
+            targetLabels,
+            testsToRunLabels,
             targets.hasError(),
             expandedResult.hasError(),
-            filteredTargets,
-            testFilteredTargets,
-            removedTargets,
+            removedTargetLabels,
             getWorkspaceName(eventHandler));
 
     // This is the same code as SkyframeLoadingPhaseRunner.
     eventHandler.post(
         new TargetParsingCompleteEvent(
             targets.getTargets(),
-            patternParsingValue.getFilteredTargets(),
-            patternParsingValue.getTestFilteredTargets(),
+            filteredTargets,
+            testFilteredTargets,
             targetPatterns,
-            patternParsingValue.getTargets()));
+            expandedTargetsToLoad));
     eventHandler.post(new TargetParsingPhaseTimeEvent(targetPatternEvalTime));
     if (callback != null) {
-      callback.notifyTargets(patternParsingValue.getTargets());
+      callback.notifyTargets(expandedTargetsToLoad);
     }
+
     eventHandler.post(
         new LoadingPhaseCompleteEvent(
-            patternParsingValue.getTargets(),
-            patternParsingValue.getRemovedTargets(),
+            expandedTargetsToLoad,
+            removedTargets,
             packageManager.getAndClearStatistics(),
             testSuiteTime));
     logger.info("Target pattern evaluation finished");
-    return patternParsingValue.toLoadingResult();
+    return patternParsingValue.toLoadingResult(eventHandler, packageManager);
   }
 
   private ResolvedTargets<Target> expandTestSuites(