[llvm] r211311 - Include Threading.h instead of forward declaring a function.

Zachary Turner zturner at google.com
Thu Jun 19 13:20:03 PDT 2014


Author: zturner
Date: Thu Jun 19 15:20:03 2014
New Revision: 211311

URL: http://llvm.org/viewvc/llvm-project?rev=211311&view=rev
Log:
Include Threading.h instead of forward declaring a function.

Previously this led to a circular header dependency, but a recent
change has since removed this dependency, so the correct fix is
to simply include the header rather than forward declare.

Modified:
    llvm/trunk/include/llvm/Support/Mutex.h

Modified: llvm/trunk/include/llvm/Support/Mutex.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Support/Mutex.h?rev=211311&r1=211310&r2=211311&view=diff
==============================================================================
--- llvm/trunk/include/llvm/Support/Mutex.h (original)
+++ llvm/trunk/include/llvm/Support/Mutex.h Thu Jun 19 15:20:03 2014
@@ -15,13 +15,11 @@
 #define LLVM_SUPPORT_MUTEX_H
 
 #include "llvm/Support/Compiler.h"
+#include "llvm/Support/Threading.h"
 #include <cassert>
 
 namespace llvm
 {
-  // Forward declare this function.
-  bool llvm_is_multithreaded();
-
   namespace sys
   {
     /// @brief Platform agnostic Mutex class.





More information about the llvm-commits mailing list