ManagedStatic: remove from GDBRegistrationListener
authorNicolai Hähnle <nicolai.haehnle@amd.com>
Sun, 10 Jul 2022 08:25:20 +0000 (10:25 +0200)
committerNicolai Hähnle <nicolai.haehnle@amd.com>
Sun, 10 Jul 2022 12:13:16 +0000 (14:13 +0200)
commitc4ccf608c2380796d64db108eed3b24c1b0140ef
tree8f8f2d59881415c882d50e1d9961a650d3849c57
parentd55985789b38dde354235d6899b2f117a3cfe344
ManagedStatic: remove from GDBRegistrationListener

An earlier version of this change originally landed as part of
e6f1f062457c928c18a88c612f39d9e168f65a85 (D129120), which caused a
Fuchsia buildbot regression in ExecutionEngine tests.

Careful review suggests that the issue was that in the earlier version,
the destructor of the JITDebugLock was run before the destructor of
GDBJITRegistrationListener. The new version of the change moves the lock
to a member variable of the (singleton!) GDBJITRegistartionListener so
that destructors are run in the right order.
llvm/lib/ExecutionEngine/GDBRegistrationListener.cpp