<br><br>
<div class="gmail_quote">On Thu, May 21, 2009 at 8:25 AM, Owen Anderson <span dir="ltr"><<a href="mailto:resistor@mac.com">resistor@mac.com</a>></span> wrote:<br>
<blockquote style="BORDER-LEFT: #ccc 1px solid; MARGIN: 0px 0px 0px 0.8ex; PADDING-LEFT: 1ex" class="gmail_quote">
<div class="im"><br>On May 20, 2009, at 5:03 PM, Howard Su wrote:<br><br>
<blockquote style="BORDER-LEFT: #ccc 1px solid; MARGIN: 0px 0px 0px 0.8ex; PADDING-LEFT: 1ex" class="gmail_quote">don't inlcude platform dependent header here. please include "llvm/Support/DataTypes.h" instead.<br>
</blockquote><br></div>That is not sufficient.  It needs windows.h in order to get the MemoryBarrier() macro.  Plus, files in lib/System/ are allowed to include platform dependent headers.  Take a look at Windows/Mutex.inc, for instance. </blockquote>

<div>Sorry I didn't express myself clearly. I mean you can not include stdint.h in atomic.h since windows don't have it. In the atomic header file, what we need it uint_t define which is in  "llvm/Support/DataTypes.h"</div>

<blockquote style="BORDER-LEFT: #ccc 1px solid; MARGIN: 0px 0px 0px 0.8ex; PADDING-LEFT: 1ex" class="gmail_quote"><span></span>
<div class="im"><br><br>
<blockquote style="BORDER-LEFT: #ccc 1px solid; MARGIN: 0px 0px 0px 0.8ex; PADDING-LEFT: 1ex" class="gmail_quote">Also, please update CMakeLists file also to add Atomic.cpp.<br></blockquote><br></div>I don't speak CMake.  Please send a patch for it.<br>
<font color="#888888"></font></blockquote>
<div> </div>
<div>Index: include/llvm/System/Atomic.h<br>===================================================================<br>--- include/llvm/System/Atomic.h (revision 72198)<br>+++ include/llvm/System/Atomic.h (working copy)<br>@@ -14,7 +14,7 @@<br>
 #ifndef LLVM_SYSTEM_ATOMIC_H<br> #define LLVM_SYSTEM_ATOMIC_H<br> <br>-#include <stdint.h><br>+#include "llvm/Support/DataTypes.h" <br> <br> namespace llvm {<br>   namespace sys {<br>Index: lib/System/CMakeLists.txt<br>
===================================================================<br>--- lib/System/CMakeLists.txt (revision 72198)<br>+++ lib/System/CMakeLists.txt (working copy)<br>@@ -1,5 +1,6 @@<br> add_llvm_library(LLVMSystem<br>   Alarm.cpp<br>
+  Atomic.cpp<br>   Disassembler.cpp<br>   DynamicLibrary.cpp<br>   Host.cpp<br></div>
<blockquote style="BORDER-LEFT: #ccc 1px solid; MARGIN: 0px 0px 0px 0.8ex; PADDING-LEFT: 1ex" class="gmail_quote"><font color="#888888"><span></span><br>--Owen<br><br></font><br>_______________________________________________<br>
llvm-commits mailing list<br><a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br><br clear="all">
<div></div><br>-- <br>-Howard<br>