diff options
| author |  janakr <janakr@google.com> | 2018-01-11 14:02:35 -0800 |
|---|---|---|
| committer |  Copybara-Service <copybara-piper@google.com> | 2018-01-11 14:04:37 -0800 |
| commit | 573807d4e9d1b7a8b6956278773dfc53b544093f (patch) | |
| tree | ce89c3a760afbb113f78621a2b1ef0cbb9cef5e5 /src/main/java/com/google/devtools/build/lib/skyframe/ArtifactFunction.java | |
| parent | 42623f59fdd3bfbdfc21490c69f21537fa32011c (diff) | |
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
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/skyframe/ArtifactFunction.java')
| -rw-r--r-- | src/main/java/com/google/devtools/build/lib/skyframe/ArtifactFunction.java | 4 |
1 files changed, 2 insertions, 2 deletions
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, |