Skip to content

Commit 31422d0

Browse files
Josh Petersonjoncham
authored andcommitted
Make the global break check volatile
The Suspend debugger test was hanging intermittently on most platforms. These seemed to occur because the codegen checks for `g_Il2CppDebuggerCheckPointEnabled` were not getting the proper value, and assuming the check point was not set, so the VM would not suspend. This meant that the main thread could not be suspended when the debugger was trying to exit, so the main thread kept looping forever, with out allowing the debugger to exit it.
1 parent 3b831f7 commit 31422d0

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

mono/mini/debugger-agent.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1161,7 +1161,7 @@ void mono_debugger_install_runtime_callbacks(MonoDebuggerRuntimeCallbacks* cbs)
11611161
}
11621162

11631163
uint32_t* g_unity_check;
1164-
void mono_debugger_install_sequence_point_check(uint32_t* check)
1164+
void mono_debugger_install_sequence_point_check(volatile uint32_t* check)
11651165
{
11661166
g_unity_check = check;
11671167
}

0 commit comments

Comments
 (0)