fix: remove racy call to isolate->IsExecutionTerminating()

leonm1 · leonm1 · commit 98a2b05c6095 · 2025-07-21T10:40:48.000-04:00
Signed-off-by: Matt Leon &lt;mattleon@google.com&gt;
diff --git a/src/v8/v8.cc b/src/v8/v8.cc
@@ -703,9 +703,6 @@ void V8::terminate() {
   auto *store_impl = reinterpret_cast<wasm::StoreImpl *>(store_.get());
   auto *isolate = store_impl->isolate();
   isolate->TerminateExecution();
-  while (isolate->IsExecutionTerminating()) {
-    std::this_thread::yield();
-  }
 }
 
 std::string V8::getFailMessage(std::string_view function_name, wasm::own<wasm::Trap> trap) {

Original file line number	Diff line number	Diff line change
`@@ -703,9 +703,6 @@ void V8::terminate() {`
`703`	`703`	`auto store_impl = reinterpret_cast<wasm::StoreImpl >(store_.get());`
`704`	`704`	`auto *isolate = store_impl->isolate();`
`705`	`705`	`isolate->TerminateExecution();`
`706`		`- while (isolate->IsExecutionTerminating()) {`
`707`		`- std::this_thread::yield();`
`708`		`- }`
`709`	`706`	`}`
`710`	`707`
`711`	`708`	`std::string V8::getFailMessage(std::string_view function_name, wasm::own<wasm::Trap> trap) {`