<div dir="rtl"><div dir="ltr">I see now, LLVM has its own Windows.h.  That's confusing, LLVM should not use standard header file names. This file should be "Windows-LLVM.h" or something else unique.</div><div dir="ltr">

<br></div><div dir="ltr">Yaron</div><div dir="ltr"><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote"><div dir="ltr">2014-02-12 11:45 GMT+02:00 NAKAMURA Takumi <span dir="ltr"><<a href="mailto:geek4civic@gmail.com" target="_blank">geek4civic@gmail.com</a>></span>:</div>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">2014-02-12 17:28 GMT+09:00 Yaron Keren <<a href="mailto:yaron.keren@gmail.com">yaron.keren@gmail.com</a>>:<br>
<div class="">> Hi,<br>
><br>
> While at it, #include "Windows.h" should be #include <Windows.h>.<br>
<br>
</div>No. "llvm/lib/Support/Windows/Windows.h" should satisfy there.<br>
I'll apply Alex's after bots became green. I think it's reasonable.<br>
<span class="HOEnZb"><font color="#888888"><br>
...Takumi<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
><br>
> Yaron<br>
><br>
><br>
><br>
> 2014-02-12 10:18 GMT+02:00 Alex Crichton <<a href="mailto:alex@crichton.co">alex@crichton.co</a>>:<br>
>><br>
>> I found that swapping the order of some header files helped fix a<br>
>> build issue that we're seeing on mingw32. Without the swap, windows.h<br>
>> was being included before _WIN32_WINNT was being defined and the<br>
>> CreateHardLinkW function was #ifdef'd out.<br>
>><br>
>> It looks like the header is mainly used to get the SHGetFolderPathW<br>
>> function, so I don't think that there'll be much fallout from the<br>
>> switch.<br>
>><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>
><br>
><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>
</div></div></blockquote></div><br></div>