Small target cleanup

Author: Kobzol

collector/src/compile/benchmark/target.rs

@@ -40,10 +40,6 @@ impl Target {
         }
     }
 
-    pub fn all() -> Vec<Self> {
-        vec![Self::X86_64UnknownLinuxGnu, Self::AArch64UnknownLinuxGnu]
-    }
-
     #[cfg(target_arch = "x86_64")]
     pub fn host() -> Self {
         Self::X86_64UnknownLinuxGnu

collector/src/compile/execute/bencher.rs

@@ -96,16 +96,6 @@ impl<'a> BenchProcessor<'a> {
         target: Target,
         stats: Stats,
     ) {
-        let backend = match backend {
-            CodegenBackend::Llvm => database::CodegenBackend::Llvm,
-            CodegenBackend::Cranelift => database::CodegenBackend::Cranelift,
-        };
-
-        let target = match target {
-            Target::X86_64UnknownLinuxGnu => database::Target::X86_64UnknownLinuxGnu,
-            Target::AArch64UnknownLinuxGnu => database::Target::AArch64UnknownLinuxGnu,
-        };
-
         let mut buf = FuturesUnordered::new();
         for (stat, value) in stats.iter() {
             buf.push(self.conn.record_statistic(
@@ -114,8 +104,8 @@ impl<'a> BenchProcessor<'a> {
                 self.benchmark.0.as_str(),
                 profile,
                 scenario,
-                backend,
-                target,
+                backend.into(),
+                target.into(),
                 stat,
                 value,
             ));

database/src/lib.rs

@@ -388,6 +388,8 @@ impl Target {
         vec![Self::X86_64UnknownLinuxGnu, Self::AArch64UnknownLinuxGnu]
     }
 
+    /// Targets that will be benchmarked by default in benchmark requests when no explicit target
+    /// is specified.
     pub fn default_targets() -> Vec<Self> {
         vec![Self::X86_64UnknownLinuxGnu]
     }