diff options
| author | Jonas Devlieghere <jonas@devlieghere.com> | 2019-08-30 15:56:14 +0000 |
|---|---|---|
| committer | Jonas Devlieghere <jonas@devlieghere.com> | 2019-08-30 15:56:14 +0000 |
| commit | a053ae0faeabc0bfb9ae72bfba6c9bdae49457d3 (patch) | |
| tree | 8a9dc8f8de11d708e743356afec45f1d2c2701b2 | |
| parent | becbdc66dcb5df6685831257ad6a5d1f22cd332b (diff) | |
| download | bcm5719-llvm-a053ae0faeabc0bfb9ae72bfba6c9bdae49457d3.tar.gz bcm5719-llvm-a053ae0faeabc0bfb9ae72bfba6c9bdae49457d3.zip | |
[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
| -rw-r--r-- | lldb/lit/Suite/lit.cfg | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lldb/lit/Suite/lit.cfg b/lldb/lit/Suite/lit.cfg index 80f5ef0a3cd..aec9690662f 100644 --- a/lldb/lit/Suite/lit.cfg +++ b/lldb/lit/Suite/lit.cfg @@ -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(): |
