<div dir="ltr"><div>I may be misunderstanding, but:</div>It seems to me that, most likely, you don't want INCLUDE, since you want to be sure of all the headers you include (and tests depend on a minimum set of headers).<div><br></div><div>But you want LIB (on Windows targetting Windows) since that's where you can find kernel32.dll and others.</div><div><br></div><div>LLVM tests only run on the host, so we don't have the “problem” of Windows targetting some other platform. Because of that, we can simply forward LIB.</div><div><br></div><div> Filipe</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Feb 19, 2015 at 9:52 AM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I'm not sure I understand what's being fixed here.<br>
<br>
LLVM's test/lit.cfg seems to just set LIB unconditionally (test/lit.cfg:75).<br>
<br>
A little higher up in this file we're conditionally putting INCLUDE in possibly_dangerous_env_vars, which seems to suggest that it that one also gets propagated through (but I can't see where that happens). Shouldn't we have the same logic for INCLUDE and LIB?<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="http://reviews.llvm.org/D7739" target="_blank">http://reviews.llvm.org/D7739</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</div></div></blockquote></div><br></div></div>