[PATCH] Remove support for runtime multithreading

Chandler Carruth chandlerc at gmail.com
Fri Jun 13 14:55:36 PDT 2014


One more thing to split out, and I think this will actually look good.

================
Comment at: lib/Support/ManagedStatic.cpp:27
@@ -25,3 +26,3 @@
   if (llvm_is_multithreaded()) {
-    llvm_acquire_global_lock();
+    llvm::MutexGuard Lock(llvm_get_global_lock());
 
----------------
This (and the related change to the interface) is the other unrelated cleanup that I would like to see separated out from the change which has policy effect.

Consider, if we need to revert the runtime control of multithreading for a brief period because something breaks, we shouldn't also lose this cleanup change.

http://reviews.llvm.org/D4139






More information about the llvm-commits mailing list