<div dir="ltr">This is completed as of <a href="http://llvm.org/viewvc/llvm-project?view=revision&revision=331224">http://llvm.org/viewvc/llvm-project?view=revision&revision=331224</a></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Apr 2, 2018 at 2:30 PM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<div><br></div><div>LLVM_ON_WIN32 is true exactly when the compiler-set _WIN32 is set. I'd like to just replace LLVM_ON_WIN32 with just _WIN32 everywhere -- _WIN32 doesn't require a custom header file, and we already are inconsistent about which of the two versions we should use in the codebase.<br></div><div><br></div><div>Let me know if you see any problems with this plan, else I'll do the switch in a week or so.</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Nico</div></font></span></div>
</blockquote></div><br></div>