Do not store metrics from self-profile runs into the DB

Author: Kobzol

collector/src/compile/execute/bencher.rs

@@ -97,6 +97,9 @@ impl<'a> BenchProcessor<'a> {
             CodegenBackend::Cranelift => database::CodegenBackend::Cranelift,
         };
 
+        // If the gathered metrics were produced with self profile enabled, then they are not
+        // realistic. Do not store the metrics into the DB for self-profile runs to avoid unnecessary
+        // DB storage.
         if let Some(files) = stats.2 {
             if env::var_os("RUSTC_PERF_UPLOAD_TO_S3").is_some() {
                 // FIXME: Record codegen backend in the self profile name
@@ -128,23 +131,23 @@ impl<'a> BenchProcessor<'a> {
                     )
                     .await;
             }
-        }
+        } else {
+            let mut buf = FuturesUnordered::new();
+            for (stat, value) in stats.0.iter() {
+                buf.push(self.conn.record_statistic(
+                    collection,
+                    self.artifact_row_id,
+                    self.benchmark.0.as_str(),
+                    profile,
+                    scenario,
+                    backend,
+                    stat,
+                    value,
+                ));
+            }
 
-        let mut buf = FuturesUnordered::new();
-        for (stat, value) in stats.0.iter() {
-            buf.push(self.conn.record_statistic(
-                collection,
-                self.artifact_row_id,
-                self.benchmark.0.as_str(),
-                profile,
-                scenario,
-                backend,
-                stat,
-                value,
-            ));
+            while let Some(()) = buf.next().await {}
         }
-
-        while let Some(()) = buf.next().await {}
     }
 
     pub async fn measure_rustc(&mut self, toolchain: &Toolchain) -> anyhow::Result<()> {