[llvm-commits] [llvm] r73639 - /llvm/trunk/lib/VMCore/PassManager.cpp

[Owen Anderson]

Author: resistor
Date: Wed Jun 17 16:28:54 2009
New Revision: 73639

URL: http://llvm.org/viewvc/llvm-project?rev=73639&view=rev
Log:
Guard mutation of the timing info global.

Modified:
    llvm/trunk/lib/VMCore/PassManager.cpp

Modified: llvm/trunk/lib/VMCore/PassManager.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/VMCore/PassManager.cpp?rev=73639&r1=73638&r2=73639&view=diff

==============================================================================
--- llvm/trunk/lib/VMCore/PassManager.cpp (original)
+++ llvm/trunk/lib/VMCore/PassManager.cpp Wed Jun 17 16:28:54 2009
@@ -20,6 +20,8 @@
 #include "llvm/Support/Streams.h"
 #include "llvm/Support/ManagedStatic.h"
 #include "llvm/Support/raw_ostream.h"
+#include "llvm/Support/Threading.h"
+#include "llvm/System/Mutex.h"
 #include "llvm/Analysis/Dominators.h"
 #include "llvm-c/Core.h"
 #include <algorithm>
@@ -355,6 +357,9 @@
 /// amount of time each pass takes to execute.  This only happens when
 /// -time-passes is enabled on the command line.
 ///
+
+static ManagedStatic<sys::Mutex> TimingInfoMutex;
+
 class VISIBILITY_HIDDEN TimingInfo {
   std::map<Pass*, Timer> TimingData;
   TimerGroup TG;
@@ -379,18 +384,22 @@
     if (dynamic_cast<PMDataManager *>(P)) 
       return;
 
+    if (llvm_is_multithreaded()) TimingInfoMutex->acquire();
     std::map<Pass*, Timer>::iterator I = TimingData.find(P);
     if (I == TimingData.end())
       I=TimingData.insert(std::make_pair(P, Timer(P->getPassName(), TG))).first;
     I->second.startTimer();
+    if (llvm_is_multithreaded()) TimingInfoMutex->release();
   }
   void passEnded(Pass *P) {
     if (dynamic_cast<PMDataManager *>(P)) 
       return;
 
+    if (llvm_is_multithreaded()) TimingInfoMutex->acquire();
     std::map<Pass*, Timer>::iterator I = TimingData.find(P);
     assert(I != TimingData.end() && "passStarted/passEnded not nested right!");
     I->second.stopTimer();
+    if (llvm_is_multithreaded()) TimingInfoMutex->release();
   }
 };