<div dir="rtl"><div dir="ltr">Re-reading, the problem is with using #ifdef inside the conditional or using the #ifdef at all?</div><div dir="ltr"><br></div></div><div class="gmail_extra"><div dir="ltr"><br><div class="gmail_quote">2015-07-13 22:33 GMT+03:00 Yaron Keren <span dir="ltr"><<a href="mailto:yaron.keren@gmail.com" target="_blank">yaron.keren@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 .8ex;border-left:1px #ccc solid;border-right:1px #ccc solid;padding-left:1ex;padding-right:1ex"><div dir="rtl"><div dir="ltr">Sure.</div><div dir="ltr">What is the proper way to write code that depends on clang host without using the #defines?</div><div dir="ltr"><br></div><div dir="ltr"><br></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote"><div dir="ltr">2015-07-13 19:57 GMT+03:00 Joerg Sonnenberger <span dir="ltr"><<a href="mailto:joerg@britannica.bec.de" target="_blank">joerg@britannica.bec.de</a>></span>:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, Jul 13, 2015 at 06:54:50PM +0300, Yaron Keren wrote:<br>
> --- tools/clang/lib/Driver/MinGWToolChain.cpp (revision 242033)<br>
> +++ tools/clang/lib/Driver/MinGWToolChain.cpp (working copy)<br>
> @@ -24,8 +24,10 @@<br>
>      : ToolChain(D, Triple, Args) {<br>
>    getProgramPaths().push_back(getDriver().getInstalledDir());<br>
><br>
> +  llvm::SmallString<1024> LibDir;<br>
>    if (getDriver().SysRoot.size())<br>
>      Base = getDriver().SysRoot;<br>
> +#ifdef LLVM_ON_WIN32<br>
>    else if (llvm::ErrorOr<std::string> GPPName =<br>
<br>
Please don't introduce such conditionals.<br>
<span><font color="#888888"><br>
Joerg<br>
</font></span></blockquote></div><br></div>
</div></div></blockquote></div></div></div>