maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[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.

llvm-svn: 370483
This commit is contained in:
Jonas Devlieghere 2019-08-30 15:56:14 +00:00
parent becbdc66dc
commit a053ae0fae
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lldb/lit/Suite/lit.cfg
View File

@ -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():