Author: jdevlieghere
Date: Fri Aug 30 08:56:14 2019
New Revision: 370483
URL: http://llvm.org/viewvc/llvm-project?rev=370483&view=rev
Log:
[lit] Fix my earlier bogus fix to not set DYLD_LIBRARY_PATH with Asan.
My follow-up commit to mess with DYLD_LIBRARY_PATH was bogus for two
reasons:
- The condition was inverted.
- We were checking the OS's environment, instead of the config's.
Two wrongs don't make a right, but the second mistake meant that the
sanitizer bot passed.
Modified:
lldb/trunk/lit/Suite/lit.cfg
Modified: lldb/trunk/lit/Suite/lit.cfg
URL:
http://llvm.org/viewvc/llvm-project/lldb/trunk/lit/Suite/lit.cfg?rev=370483&r1=370482&r2=370483&view=diff
==============================================================================
--- lldb/trunk/lit/Suite/lit.cfg (original)
+++ lldb/trunk/lit/Suite/lit.cfg Fri Aug 30 08:56:14 2019
@@ -39,7 +39,7 @@ def find_shlibpath_var():
elif platform.system() == 'Windows':
yield 'PATH'
-if 'DYLD_INSERT_LIBRARIES' in os.environ:
+if not 'DYLD_INSERT_LIBRARIES' in config.environment:
# Shared library build of LLVM may require LD_LIBRARY_PATH or equivalent.
# This clashes with DYLD_INSERT_LIBRARIES which is needed on Darwin.
for shlibpath_var in find_shlibpath_var():
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
