Include Threading.h instead of forward declaring a function.

Zachary Turner · Zachary Turner · commit e0d89ff76414 · 2014-06-19T20:20:03.000Z
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. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@211311 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/include/llvm/Support/Mutex.h b/include/llvm/Support/Mutex.h
@@ -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.

Original file line number	Diff line number	Diff line change
`@@ -15,13 +15,11 @@`
`15`	`15`	`#define LLVM_SUPPORT_MUTEX_H`
`16`	`16`
`17`	`17`	`#include "llvm/Support/Compiler.h"`
	`18`	`+#include "llvm/Support/Threading.h"`
`18`	`19`	`#include <cassert>`
`19`	`20`
`20`	`21`	`namespace llvm`
`21`	`22`	`{`
`22`		`- // Forward declare this function.`
`23`		`- bool llvm_is_multithreaded();`
`24`		`-`
`25`	`23`	`namespace sys`
`26`	`24`	`{`
`27`	`25`	`/// @brief Platform agnostic Mutex class.`