<div dir="ltr">Good to know. I'll work on that right now. Thanks!</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Il giorno mer 27 gen 2021 alle ore 11:41 Stella Stamenova via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> ha scritto:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">stella.stamenova added inline comments.<br>
<br>
<br>
================<br>
Comment at: lldb/tools/lldb-vscode/FifoFiles.cpp:9<br>
+<br>
+#if !defined(WIN32)<br>
+#include <sys/stat.h><br>
----------------<br>
Also, this is no good. It works if you are targeting windows on windows, but not if you are targeting something else and building on windows. There are a few different ways this is done in LLVM/clang and they are generally not based on defined(WIN32). Here are a couple of examples:<br>
<br>
>From llvm\lib\Support\Path.cpp:<br>
<br>
```<br>
#if !defined(_MSC_VER) && !defined(__MINGW32__)<br>
```<br>
<br>
>From clang\lib\Driver\Driver.cpp:<br>
<br>
```<br>
#if LLVM_ON_UNIX<br>
```<br>
<br>
>From llvm\lib\Support\ErrorHandling.cpp:<br>
<br>
```<br>
#if defined(HAVE_UNISTD_H)<br>
```<br>
<br>
I suggest browsing through the code and finding the most appropriate way to manage your includes. In the mean time, this is breaking our internal builds that run on Windows but do not target Windows.<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D93951/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D93951/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D93951" rel="noreferrer" target="_blank">https://reviews.llvm.org/D93951</a><br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>- Walter Erquínigo Pezo<br><br></div></div></div>