<div dir="auto">This is great to know. Til. I'll change all of those macros to _WIN32 for consistency then</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Il Gio 4 Feb 2021, 3:25 AM Martin Storsjö <<a href="mailto:martin@martin.st" target="_blank" rel="noreferrer">martin@martin.st</a>> ha scritto:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, 28 Jan 2021, Walter Erquinigo via lldb-commits wrote:<br>
<br>
><br>
> Author: Walter Erquinigo<br>
> Date: 2021-01-28T09:36:13-08:00<br>
> New Revision: 0bca9a7ce2eeaa9f1d732ffbc17769560a2b236e<br>
><br>
> URL: <a href="https://github.com/llvm/llvm-project/commit/0bca9a7ce2eeaa9f1d732ffbc17769560a2b236e" rel="noreferrer noreferrer noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/0bca9a7ce2eeaa9f1d732ffbc17769560a2b236e</a><br>
> DIFF: <a href="https://github.com/llvm/llvm-project/commit/0bca9a7ce2eeaa9f1d732ffbc17769560a2b236e.diff" rel="noreferrer noreferrer noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/0bca9a7ce2eeaa9f1d732ffbc17769560a2b236e.diff</a><br>
><br>
> LOG: Fix lldb-vscode builds on Windows targeting POSIX<br>
><br>
> @stella.stamenova found out that lldb-vscode's Win32 macros were failing<br>
> when building on windows targetings POSIX platforms.<br>
><br>
> I'm changing these macros for LLVM_ON_UNIX, which should be more<br>
> accurate.<br>
<br>
Just for the record, the reason for things failing wasn't so much that <br>
LLVM_ON_UNIX is more accurate than the macros that the code tried to use <br>
before. Before, the code tried to check the define "WIN32" in some files, <br>
and "_WIN32" in others (and one case even used the macro "WIN_32").<br>
<br>
"_WIN32" is the correct one to check, it's predefined by the compilers <br>
when targeting windows. Most build tools don't normally define "WIN32" <br>
without a leading underscore (although e.g. some visual studio project <br>
file templates include it in the list of user-defined macros). So that's <br>
why things broke before.<br>
<br>
Btw, the checks like "#if LLVM_ON_UNIX" are a bit incorrect too, they <br>
should be "#ifdef LLVM_ON_UNIX", as the macro is entirely undefined for <br>
non-unix platforms.<br>
<br>
These commits should be backported to the 12.x release branch too, because <br>
building lldb for windows (or in particular, lldb-vscode) is broken there <br>
right now.<br>
<br>
// Martin<br>
<br>
</blockquote></div>