[CI] Added Serial Execution Engine Test

Since the CI always installs oneTBB and the execution engine is set to
auto, I found that the CI always tested with the tbb execution engine.
Some users may not have oneTBB installed for one reason or another and
we need to ensure that VTR always builds.

Added a CI test which sets the parallel execution engine to serial for
Tatum and VPR.

Author: AlexandreSinger

.github/workflows/test.yml

@@ -199,6 +199,12 @@ jobs:
             suite: 'vtr_reg_basic',
             extra_pkgs: ""
           },
+          {
+            name: 'Basic with serial execution engine (no TBB)',
+            params: '-DCMAKE_COMPILE_WARNING_AS_ERROR=on -DVTR_IPO_BUILD=off -DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on -DVPR_EXECUTION_ENGINE=serial -DTATUM_EXECUTION_ENGINE=serial',
+            suite: 'vtr_reg_basic',
+            extra_pkgs: ""
+          },
           {
             name: 'Basic with VTR_ENABLE_DEBUG_LOGGING',
             params: '-DCMAKE_COMPILE_WARNING_AS_ERROR=on -DVTR_IPO_BUILD=off -DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on -DVTR_ENABLE_DEBUG_LOGGING=on',

vpr/src/route/route_utils.cpp

@@ -19,10 +19,7 @@
 #include "route_tree.h"
 #include "rr_graph.h"
 #include "tatum/TimingReporter.hpp"
-
-#ifdef VPR_USE_TBB
 #include "stats.h"
-#endif // VPR_USE_TBB
 
 bool check_net_delays(const Netlist<>& net_list, NetPinsMatrix<float>& net_delay) {
     constexpr float ERROR_TOL = 0.0001;