Fix bug that all full clean targets are cleaned.

Only the first target should be cleaned or later incremental build is not incremental.

--
PiperOrigin-RevId: 150096210
MOS_MIGRATED_REVID=150096210

1 files changed, 5 insertions, 5 deletions

diff --git a/src/tools/benchmark/java/com/google/devtools/build/benchmark/BuildGroupRunner.java b/src/tools/benchmark/java/com/google/devtools/build/benchmark/BuildGroupRunner.java
index 82d4d3b490..aedd7645bc 100644
--- a/src/tools/benchmark/java/com/google/devtools/build/benchmark/BuildGroupRunner.java
+++ b/src/tools/benchmark/java/com/google/devtools/build/benchmark/BuildGroupRunner.java
@@ -112,6 +112,11 @@ class BuildGroupRunner {
     }
     lastIsIncremental = envConfig.getIncremental();
 
+    // Builder's clean method, only clean before the first target
+    if (targetIndex == 0 && envConfig.getCleanBeforeBuild()) {
+      builder.clean();
+    }
+
     if (removeFirstResult) {
       buildTargetAndGetElapsedTime(buildBinary, envConfig, targetConfig);
     }
@@ -129,11 +134,6 @@ class BuildGroupRunner {
   private double buildTargetAndGetElapsedTime(
       Path buildBinary, BuildEnvConfig envConfig, BuildTargetConfig targetConfig)
       throws CommandException {
-    // Builder's clean method
-    if (envConfig.getCleanBeforeBuild()) {
-      builder.clean();
-    }
-
     // Run build
     double elapsedTime =
         builder.buildAndGetElapsedTime(