Internal change

PiperOrigin-RevId: 169631693

1 files changed, 7 insertions, 4 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/actions/Artifact.java b/src/main/java/com/google/devtools/build/lib/actions/Artifact.java
index 1bde580120..9e05d2ec44 100644
--- a/src/main/java/com/google/devtools/build/lib/actions/Artifact.java
+++ b/src/main/java/com/google/devtools/build/lib/actions/Artifact.java
@@ -156,6 +156,7 @@ public class Artifact
   public static final Predicate<Artifact> MIDDLEMAN_FILTER = input -> !input.isMiddlemanArtifact();
 
   private final int hashCode;
+  private final Path path;
   private final Root root;
   private final PathFragment execPath;
   private final PathFragment rootRelativePath;
@@ -189,6 +190,7 @@ public class Artifact
           + " (root: " + root + ")");
     }
     this.hashCode = path.hashCode();
+    this.path = path;
     this.root = root;
     this.execPath = execPath;
     // These two lines establish the invariant that
@@ -244,7 +246,7 @@ public class Artifact
   }
 
   public final Path getPath() {
-    return root.getPath().getRelative(rootRelativePath);
+    return path;
   }
 
   public boolean hasParent() {
@@ -587,9 +589,10 @@ public class Artifact
     if (!(other instanceof Artifact)) {
       return false;
     }
+    // We don't bother to check root in the equivalence relation, because we
+    // assume that no root is an ancestor of another one.
     Artifact that = (Artifact) other;
-    return Objects.equals(this.rootRelativePath, that.rootRelativePath)
-        && Objects.equals(this.root, that.root);
+    return Objects.equals(this.path, that.path);
   }
 
   @Override
@@ -623,7 +626,7 @@ public class Artifact
       return "[" + root + "]" + rootRelativePath;
     } else {
       // Derived Artifact: path and root are under execRoot
-      PathFragment execRoot = trimTail(getPath().asFragment(), execPath);
+      PathFragment execRoot = trimTail(path.asFragment(), execPath);
       return "[[" + execRoot + "]" + root.getPath().asFragment().relativeTo(execRoot) + "]"
           + rootRelativePath;
     }