diff options
author | NAKAMURA Takumi <geek4civic@gmail.com> | 2014-10-14 15:58:16 +0000 |
---|---|---|
committer | NAKAMURA Takumi <geek4civic@gmail.com> | 2014-10-14 15:58:16 +0000 |
commit | 256d37ad31c49e1eabcf4af02c09b479953f1101 (patch) | |
tree | 8e74c0ecbb644c96de51f9d2d81d8993075df619 /llvm/lib/Support/ManagedStatic.cpp | |
parent | 1a77f6664eb268e3131a87c78ef818db41d2a589 (diff) | |
download | llvm-256d37ad31c49e1eabcf4af02c09b479953f1101.zip llvm-256d37ad31c49e1eabcf4af02c09b479953f1101.tar.gz llvm-256d37ad31c49e1eabcf4af02c09b479953f1101.tar.bz2 |
Revert r219638, (r219640 and r219676), "Removing the static destructor from ManagedStatic.cpp by controlling the allocation and de-allocation of the mutex."
It caused hang-up on msc17 builder, probably deadlock.
llvm-svn: 219687
Diffstat (limited to 'llvm/lib/Support/ManagedStatic.cpp')
-rw-r--r-- | llvm/lib/Support/ManagedStatic.cpp | 23 |
1 files changed, 6 insertions, 17 deletions
diff --git a/llvm/lib/Support/ManagedStatic.cpp b/llvm/lib/Support/ManagedStatic.cpp index fbcce59..b8fb284 100644 --- a/llvm/lib/Support/ManagedStatic.cpp +++ b/llvm/lib/Support/ManagedStatic.cpp @@ -16,22 +16,16 @@ #include "llvm/Support/Atomic.h" #include "llvm/Support/Mutex.h" #include "llvm/Support/MutexGuard.h" -#include "llvm/Support/Threading.h" #include <cassert> using namespace llvm; static const ManagedStaticBase *StaticList = nullptr; -static sys::Mutex *ManagedStaticMutex = nullptr; -static void initializeMutex() { - ManagedStaticMutex = new sys::Mutex(); -} - -static sys::Mutex* getManagedStaticMutex() { +static sys::Mutex& getManagedStaticMutex() { // We need to use a function local static here, since this can get called // during a static constructor and we need to guarantee that it's initialized // correctly. - llvm_call_once<initializeMutex>(); + static sys::Mutex ManagedStaticMutex; return ManagedStaticMutex; } @@ -39,7 +33,7 @@ void ManagedStaticBase::RegisterManagedStatic(void *(*Creator)(), void (*Deleter)(void*)) const { assert(Creator); if (llvm_is_multithreaded()) { - MutexGuard Lock(*getManagedStaticMutex()); + MutexGuard Lock(getManagedStaticMutex()); if (!Ptr) { void* tmp = Creator(); @@ -89,13 +83,8 @@ void ManagedStaticBase::destroy() const { /// llvm_shutdown - Deallocate and destroy all ManagedStatic variables. void llvm::llvm_shutdown() { - { - MutexGuard Lock(*getManagedStaticMutex()); - - while (StaticList) - StaticList->destroy(); - } + MutexGuard Lock(getManagedStaticMutex()); - delete ManagedStaticMutex; - ManagedStaticMutex = nullptr; + while (StaticList) + StaticList->destroy(); } |