<div dir="ltr">Sounds good. Feel free to submit that patch then. Thanks!</div><br><div class="gmail_quote"><div dir="ltr">On Wed, Jan 2, 2019 at 5:12 PM Tom Tan <<a href="mailto:Tom.Tan@microsoft.com">Tom.Tan@microsoft.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div lang="EN-US" link="blue" vlink="purple">
<div class="m_4231989031983166875WordSection1">
<p class="MsoNormal">Thanks Zachary. Yes, exactly, it is that “Windows” folder needs to be removed from the include path for WindowsSupport.h because both Threading.inc and WindowsSupport.h are in the same “Windows” folder.
<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal"><b>From:</b> Zachary Turner <<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>> <br>
<b>Sent:</b> Wednesday, January 2, 2019 5:07 PM<br>
<b>To:</b> <a href="mailto:reviews%2BD30526%2Bpublic%2Bc109a870c771c88e@reviews.llvm.org" target="_blank">reviews+D30526+public+c109a870c771c88e@reviews.llvm.org</a><br>
<b>Cc:</b> <a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@gmail.com</a>; <a href="mailto:chris.bieneman@me.com" target="_blank">chris.bieneman@me.com</a>; <a href="mailto:pavel@labath.sk" target="_blank">pavel@labath.sk</a>; <a href="mailto:jfbastien@apple.com" target="_blank">jfbastien@apple.com</a>; <a href="mailto:notstina@gmail.com" target="_blank">notstina@gmail.com</a>; Tom Tan <<a href="mailto:Tom.Tan@microsoft.com" target="_blank">Tom.Tan@microsoft.com</a>>; <a href="mailto:n54@gmx.com" target="_blank">n54@gmx.com</a>; <a href="mailto:dccitaliano@gmail.com" target="_blank">dccitaliano@gmail.com</a>; <a href="mailto:danalbert@google.com" target="_blank">danalbert@google.com</a>; <a href="mailto:srhines@google.com" target="_blank">srhines@google.com</a>; <a href="mailto:emaste@freebsd.org" target="_blank">emaste@freebsd.org</a>; <a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>;
<a href="mailto:llvm@inglorion.net" target="_blank">llvm@inglorion.net</a>; <a href="mailto:sguelton@redhat.com" target="_blank">sguelton@redhat.com</a><br>
<b>Subject:</b> Re: [PATCH] D30526: [Support] Add functions to get and set thread name.<u></u><u></u></p></div></div><div lang="EN-US" link="blue" vlink="purple"><div class="m_4231989031983166875WordSection1">
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal">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.<u></u><u></u></p>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<div>
<p class="MsoNormal">On Wed, Jan 2, 2019 at 4:17 PM Tom Tan via Phabricator <<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>> wrote:<u></u><u></u></p>
</div>
<blockquote style="border:none;border-left:solid #cccccc 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-right:0in">
<p class="MsoNormal" style="margin-bottom:12.0pt">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://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Freviews.llvm.org%2FD30526%2Fnew%2F&data=02%7C01%7CTom.Tan%40microsoft.com%7Ce02a6e679c804406da1e08d67117bd4c%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636820744105460561&sdata=ugrFITdgfnnB9XROMeeEOHzYc2u3mTHKbnXYqNP6NFc%3D&reserved=0" target="_blank">
https://reviews.llvm.org/D30526/new/</a><br>
<br>
<a href="https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Freviews.llvm.org%2FD30526&data=02%7C01%7CTom.Tan%40microsoft.com%7Ce02a6e679c804406da1e08d67117bd4c%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636820744105460561&sdata=RkTzJdhb1VKBQOHl0G5yKXURkvz1nb98bughGnSw3oo%3D&reserved=0" target="_blank">https://reviews.llvm.org/D30526</a><br>
<br>
<br>
<u></u><u></u></p>
</blockquote>
</div>
</div></div></blockquote></div>