From 573807d4e9d1b7a8b6956278773dfc53b544093f Mon Sep 17 00:00:00 2001 From: janakr Date: Thu, 11 Jan 2018 14:02:35 -0800 Subject: 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 --- .../java/com/google/devtools/build/lib/skyframe/ArtifactFunction.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/main/java/com/google/devtools/build/lib/skyframe/ArtifactFunction.java') diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/ArtifactFunction.java b/src/main/java/com/google/devtools/build/lib/skyframe/ArtifactFunction.java index bb0ddbacd9..10016a1e5d 100644 --- a/src/main/java/com/google/devtools/build/lib/skyframe/ArtifactFunction.java +++ b/src/main/java/com/google/devtools/build/lib/skyframe/ArtifactFunction.java @@ -315,7 +315,7 @@ class ArtifactFunction implements SkyFunction { ArtifactOwner artifactOwner = artifact.getArtifactOwner(); Preconditions.checkState(artifactOwner instanceof ActionLookupKey, "", artifact, artifactOwner); - return ActionLookupValue.key((ActionLookupKey) artifactOwner); + return (ActionLookupKey) artifactOwner; } @Nullable @@ -326,7 +326,7 @@ class ArtifactFunction implements SkyFunction { if (value == null) { ArtifactOwner artifactOwner = artifact.getArtifactOwner(); Preconditions.checkState( - artifactOwner == CoverageReportValue.ARTIFACT_OWNER, + artifactOwner == CoverageReportValue.COVERAGE_REPORT_KEY, "Not-yet-present artifact owner: %s (%s %s)", artifactOwner, artifact, -- cgit v1.2.3