Update comments related to art and startup profiles
Change-Id: I9c587a6f63e1ecd6538b5fdd4768ddfdf52b9b6c
diff --git a/src/main/java/com/android/tools/r8/dex/ApplicationWriter.java b/src/main/java/com/android/tools/r8/dex/ApplicationWriter.java
index 8aa2096..90c8141 100644
--- a/src/main/java/com/android/tools/r8/dex/ApplicationWriter.java
+++ b/src/main/java/com/android/tools/r8/dex/ApplicationWriter.java
@@ -247,9 +247,8 @@
&& options.enableMainDexListCheck) {
distributor = new VirtualFile.MonoDexDistributor(this, classes, options);
} else {
- // Retrieve the startup order for writing the app. In R8, the startup order is created
- // up-front to guide optimizations through-out the compilation. In D8, the startup
- // order is only used for writing the app, so we create it here for the first time.
+ // Retrieve the startup order for writing the app. Use an empty startup profile if the startup
+ // profile should not be used for layout.
StartupProfile startupProfile =
options.getStartupOptions().isStartupLayoutOptimizationEnabled()
? appView.getStartupProfile()
diff --git a/src/main/java/com/android/tools/r8/profile/art/NonEmptyArtProfileCollection.java b/src/main/java/com/android/tools/r8/profile/art/NonEmptyArtProfileCollection.java
index 0174a16..4f98cdf 100644
--- a/src/main/java/com/android/tools/r8/profile/art/NonEmptyArtProfileCollection.java
+++ b/src/main/java/com/android/tools/r8/profile/art/NonEmptyArtProfileCollection.java
@@ -97,6 +97,14 @@
}
}
+ /**
+ * Processes the D8 art profiles before forwarding them to R8, in R8 partial.
+ *
+ * <p>When testing, a synthetic art profile is injected for completeness testing. This first runs
+ * the completeness checker and then removes the synthetic profile.
+ *
+ * <p>In release, this returns the art profiles as-is.
+ */
@Override
public ArtProfileCollection transformForR8Partial(AppView<AppInfo> appView) {
ArtProfileOptions artProfileOptions = appView.options().getArtProfileOptions();
