Convert ActionLookupKey implementations to directly implement SkyKey, removing the layer of indirection of getting SkyKey out of ActionLookupKey, which uses more memory for no reason.

PiperOrigin-RevId: 181658615

1 files changed, 8 insertions, 8 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/PostConfiguredTargetFunction.java b/src/main/java/com/google/devtools/build/lib/skyframe/PostConfiguredTargetFunction.java
index 779dbaa255..3ae57a524e 100644
--- a/src/main/java/com/google/devtools/build/lib/skyframe/PostConfiguredTargetFunction.java
+++ b/src/main/java/com/google/devtools/build/lib/skyframe/PostConfiguredTargetFunction.java
@@ -51,12 +51,12 @@ import javax.annotation.Nullable;
 public class PostConfiguredTargetFunction implements SkyFunction {
   private static final Function<Dependency, SkyKey> TO_KEYS =
       new Function<Dependency, SkyKey>() {
-    @Override
-    public SkyKey apply(Dependency input) {
-      return PostConfiguredTargetValue.key(
-          new ConfiguredTargetKey(input.getLabel(), input.getConfiguration()));
-    }
-  };
+        @Override
+        public SkyKey apply(Dependency input) {
+          return PostConfiguredTargetValue.key(
+              ConfiguredTargetKey.of(input.getLabel(), input.getConfiguration()));
+        }
+      };
 
   private final SkyframeExecutor.BuildViewProvider buildViewProvider;
   private final RuleClassProvider ruleClassProvider;
@@ -74,8 +74,8 @@ public class PostConfiguredTargetFunction implements SkyFunction {
       throws SkyFunctionException, InterruptedException {
     ImmutableMap<ActionAnalysisMetadata, ConflictException> badActions =
         PrecomputedValue.BAD_ACTIONS.get(env);
-    ConfiguredTargetValue ctValue = (ConfiguredTargetValue)
-        env.getValue(ConfiguredTargetValue.key((ConfiguredTargetKey) skyKey.argument()));
+    ConfiguredTargetValue ctValue =
+        (ConfiguredTargetValue) env.getValue((ConfiguredTargetKey) skyKey.argument());
     if (env.valuesMissing()) {
       return null;
     }