summaryrefslogtreecommitdiff
path: root/lib/IR
diff options
context:
space:
mode:
authorZachary Turner <zturner@google.com>2014-06-11 23:03:31 +0000
committerZachary Turner <zturner@google.com>2014-06-11 23:03:31 +0000
commita8763052319c654b924f052f55998e040a11b3f6 (patch)
tree6612d366fae070eed0392e30b67e7c8be9c0b151 /lib/IR
parent07aac43603269ab7b8f105b32e24a67d915ac01d (diff)
downloadllvm-a8763052319c654b924f052f55998e040a11b3f6.tar.gz
llvm-a8763052319c654b924f052f55998e040a11b3f6.tar.bz2
llvm-a8763052319c654b924f052f55998e040a11b3f6.tar.xz
Don't acquire the mutex during the destructor of PassRegistry.
This destructor is run as part of static program termination, and so all ManagedStatics (including this lock) will have been destroyed by llvm_shutdown. Furthermore, if there is actually a race condition during static program termination, then we are just hiding a bug somewhere else, because other threads should not be running at this point. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@210717 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/IR')
-rw-r--r--lib/IR/PassRegistry.cpp5
1 files changed, 4 insertions, 1 deletions
diff --git a/lib/IR/PassRegistry.cpp b/lib/IR/PassRegistry.cpp
index 6a5bee2f57..e85fabffb8 100644
--- a/lib/IR/PassRegistry.cpp
+++ b/lib/IR/PassRegistry.cpp
@@ -73,7 +73,10 @@ void *PassRegistry::getImpl() const {
//
PassRegistry::~PassRegistry() {
- sys::SmartScopedWriter<true> Guard(*Lock);
+ // Don't acquire the mutex here. This is destroyed during static execution of
+ // static destructors, after llvm_shutdown() has been called, so all instances
+ // of all ManagedStatics (including the Mutex), will have been destroyed as
+ // well.
PassRegistryImpl *Impl = static_cast<PassRegistryImpl*>(pImpl);
delete Impl;
pImpl = nullptr;