Dump profiler trace after flushing microtasks in CLI

Reviewed By: avp Differential Revision: D69146622 fbshipit-source-id: fe12d0ed792324961d82c2ff11b6903b0ef201d9
facebook · Feb 5, 2025 · 8b30967 · 8b30967
1 parent b92470c
commit 8b30967
Showing 1 changed file with 15 additions and 15 deletions.
diff --git a/lib/ConsoleHost/ConsoleHost.cpp b/lib/ConsoleHost/ConsoleHost.cpp
@@ -465,21 +465,6 @@ bool executeHBCBytecodeImpl(
       sourceURL,
       vm::Runtime::makeNullHandle<vm::Environment>());
 
-#if HERMESVM_SAMPLING_PROFILER_AVAILABLE
-  switch (options.sampleProfiling) {
-    case ExecuteOptions::SampleProfilingMode::None:
-      break;
-    case ExecuteOptions::SampleProfilingMode::Chrome:
-      vm::SamplingProfiler::disable();
-      runtime->samplingProfiler->dumpChromeTrace(llvh::errs());
-      break;
-    case ExecuteOptions::SampleProfilingMode::Tracery:
-      vm::SamplingProfiler::disable();
-      runtime->samplingProfiler->dumpTraceryTrace(llvh::errs());
-      break;
-  }
-#endif // HERMESVM_SAMPLING_PROFILER_AVAILABLE
-
   bool threwException = status == vm::ExecutionStatus::EXCEPTION;
 
   if (threwException) {
@@ -513,6 +498,21 @@ bool executeHBCBytecodeImpl(
     }
   }
 
+#if HERMESVM_SAMPLING_PROFILER_AVAILABLE
+  switch (options.sampleProfiling) {
+    case ExecuteOptions::SampleProfilingMode::None:
+      break;
+    case ExecuteOptions::SampleProfilingMode::Chrome:
+      vm::SamplingProfiler::disable();
+      runtime->samplingProfiler->dumpChromeTrace(llvh::errs());
+      break;
+    case ExecuteOptions::SampleProfilingMode::Tracery:
+      vm::SamplingProfiler::disable();
+      runtime->samplingProfiler->dumpTraceryTrace(llvh::errs());
+      break;
+  }
+#endif // HERMESVM_SAMPLING_PROFILER_AVAILABLE
+
 #ifdef HERMESVM_PROFILER_OPCODE
   runtime->dumpOpcodeStats(llvh::outs());
 #endif