Merge pull request #40 from colleeneb/cuda_best_flops

Modifications to avoid including "gflop/s" in the "best gflop/s" computation if the validation fails

src/main.cpp

@@ -5,11 +5,11 @@
 #include <string>
 #include <vector>
 
+#include <algorithm>
 #include <fstream>
-#include <sstream>
 #include <functional>
-#include <algorithm>
 #include <iostream>
+#include <sstream>
 #include <unordered_map>
 
 #include "bude.h"
@@ -27,8 +27,7 @@
 #endif
 
 #ifndef MINIBUDE_COMPILE_COMMANDS
-  #define MINIBUDE_COMPILE_COMMANDS                                                                                    \
-    { MINIBUDE_WARN_NOT_CMAKE }
+  #define MINIBUDE_COMPILE_COMMANDS {MINIBUDE_WARN_NOT_CMAKE}
 #endif
 
 #ifdef USE_CPU_FEATURES
@@ -388,9 +387,12 @@ std::pair<double, std::vector<size_t>> validate(const Sample &sample, const Para
     }
     // flush at the end to make sure errors are clumped together
     std::cerr << std::flush;
+    return {valid, maxDiffPct, s, SummaryStats<double>({std::numeric_limits<double>::max()}), 0, 0, 0};
+  } else if (!valid)
+    return {valid, maxDiffPct, s, SummaryStats<double>({std::numeric_limits<double>::max()}), 0, 0, 0};
+  else {
+    return {valid, maxDiffPct, s, SummaryStats<double>(msWithoutWarmup), gflops, gfinsts, interactions_per_sec};
   }
-
-  return {valid, maxDiffPct, s, SummaryStats<double>(msWithoutWarmup), gflops, gfinsts, interactions_per_sec};
 }
 
 [[nodiscard]] std::pair<int, std::string> selectDevice(const std::string &needle,