1 files changed, 16 insertions, 1 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/test/InstrumentedFilesProvider.java b/src/main/java/com/google/devtools/build/lib/rules/test/InstrumentedFilesProvider.java
index 2d06204bcc..24017da115 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/test/InstrumentedFilesProvider.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/test/InstrumentedFilesProvider.java
@@ -26,7 +26,7 @@ import java.util.Map;
 public interface InstrumentedFilesProvider extends TransitiveInfoProvider {
 
   /**
-   * Returns a collection of source files for instrumented binaries.
+   * The transitive closure of instrumented source files.
    */
   NestedSet<Artifact> getInstrumentedFiles();
 
@@ -35,6 +35,21 @@ public interface InstrumentedFilesProvider extends TransitiveInfoProvider {
    */
   NestedSet<Artifact> getInstrumentationMetadataFiles();
 
+  /**
+   * The transitive closure of instrumented source files for which baseline coverage should be
+   * generated. In general, this is a subset of the instrumented source files: it only contains
+   * instrumented source files from rules that support baseline coverage.
+   */
+  NestedSet<Artifact> getBaselineCoverageInstrumentedFiles();
+
+  /**
+   * The output artifact of the baseline coverage action; this is only ever a single artifact, which
+   * contains baseline coverage for the entire transitive closure of source files.
+   */
+  // TODO(ulfjack): Change this to a single Artifact. Also change how it's generated. It's better to
+  // generate actions such that each action only covers the source files of a single rule, in
+  // particular because baseline coverage is language-specific (it requires a parser for the
+  // specific language), and we don't want to depend on all language parsers from any single rule.
   NestedSet<Artifact> getBaselineCoverageArtifacts();
 
   /**