[PATCH] Don't set $LIB if we're not targetting windows.
Hans Wennborg
hans at chromium.org
Thu Feb 19 09:52:47 PST 2015
I'm not sure I understand what's being fixed here.
LLVM's test/lit.cfg seems to just set LIB unconditionally (test/lit.cfg:75).
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?
http://reviews.llvm.org/D7739
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list