Refactored the TreeNode::executeTick() function to use a scoped timer for performance monitoring. (#861) (#863)

Update src/tree_node.cpp

Co-authored-by: wangzheng <[email protected]>
Co-authored-by: Davide Faconti <[email protected]>

Author: 3 people

src/tree_node.cpp

@@ -106,13 +106,19 @@ NodeStatus TreeNode::executeTick()
     if(!substituted)
     {
       using namespace std::chrono;
+
       auto t1 = steady_clock::now();
+      // trick to prevent the compile from reordering the order of execution. See #861
+      // This makes sure that the code is executed at the end of this scope
+      std::shared_ptr<void> execute_later(nullptr, [&](...) {
+        auto t2 = steady_clock::now();
+        if(monitor_tick)
+        {
+          monitor_tick(*this, new_status, duration_cast<microseconds>(t2 - t1));
+        }
+      });
+
       new_status = tick();
-      auto t2 = steady_clock::now();
-      if(monitor_tick)
-      {
-        monitor_tick(*this, new_status, duration_cast<microseconds>(t2 - t1));
-      }
     }
   }