[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