<div dir="ltr">I'm on vacation right now so I don't have a source tree that I can get access to, but if the two files are in the same directory then I think we shouldn't specify the folder name, and it should just be #include "WindowsSupport.h".  Let me know if I'm misunderstood the situation though.</div><br><div class="gmail_quote"><div dir="ltr">On Wed, Jan 2, 2019 at 4:17 PM Tom Tan via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">TomTan added a comment.<br>
Herald added subscribers: kristina, jfb.<br>
<br>
Hi Zachary, does this line "#include "Windows/WindowsSupport.h" " need to be changed to "#include "WindowsSupport.h" " after it is moved from Threading.cpp to Threading.inc, because Threading.inc which includes WindowsSupport.h is under Windows folder, so no need to have it in the include path. This current include path could trigger warning "#include resolved using non-portable Microsoft search rules as: ..."<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D30526/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D30526/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D30526" rel="noreferrer" target="_blank">https://reviews.llvm.org/D30526</a><br>
<br>
<br>
<br>
</blockquote></div>