From: Chris Lattner <sabre@nondot.org>
Date: Sat, 22 Aug 2009 04:07:34 +0000 (+0000)
Subject: revert r79708 + r79711
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=895a55e66d107aa96a1e763a62a97e8ad62284a0;p=oota-llvm.git

revert r79708 + r79711


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@79720 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/CodeGen/SelectionDAG/SelectionDAG.cpp b/lib/CodeGen/SelectionDAG/SelectionDAG.cpp
index a3161b4c5e3..72f21125263 100644
--- a/lib/CodeGen/SelectionDAG/SelectionDAG.cpp
+++ b/lib/CodeGen/SelectionDAG/SelectionDAG.cpp
@@ -5017,19 +5017,17 @@ void SDNode::Profile(FoldingSetNodeID &ID) const {
 
 static ManagedStatic<std::set<EVT, EVT::compareRawBits> > EVTs;
 static EVT VTs[MVT::LAST_VALUETYPE];
+static ManagedStatic<sys::SmartMutex<true> > VTMutex;
 
 /// getValueTypeList - Return a pointer to the specified value type.
 ///
 const EVT *SDNode::getValueTypeList(EVT VT) {
+  sys::SmartScopedLock<true> Lock(*VTMutex);
   if (VT.isExtended()) {
     return &(*EVTs->insert(VT).first);
   } else {
-    // All writes to this location will have the same value, so it's ok
-    // to race on it.  We only need to ensure that at least one write has
-    // succeeded before we return the pointer into the array.
     VTs[VT.getSimpleVT().SimpleTy] = VT;
-    sys::MemoryFence();
-    return VTs + VT.getSimpleVT().SimpleTy;
+    return &VTs[VT.getSimpleVT().SimpleTy];
   }
 }