[all-commits] [llvm/llvm-project] b5c0fc: [sanitizer_common] posix_spawn test should forward...

[Andrew Haberlandt via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b5c0fcdadeed8803addf8d2aec6142fb9d8a5660
      https://github.com/llvm/llvm-project/commit/b5c0fcdadeed8803addf8d2aec6142fb9d8a5660
  Author: Andrew Haberlandt <ndrewh at users.noreply.github.com>
  Date:   2025-11-20 (Thu, 20 Nov 2025)

  Changed paths:
    M compiler-rt/test/sanitizer_common/TestCases/Posix/posix_spawn.c

  Log Message:
  -----------
  [sanitizer_common] posix_spawn test should forward DYLD_LIBRARY_PATH (#168795)

This test explicitly sets the environment for a spawned process. Without
DYLD_LIBRARY_PATH, the spawned process may use a ASAN runtime other than
the one that was used by the parent process That other runtime library
may not work at all, or may not be in the default search path. Either
case can cause the spawned process to die before it makes it to main,
thus failing the test. The compiler-rt lit config sets the library path
variable
[here](https://github.com/llvm/llvm-project/blob/main/compiler-rt/test/lit.common.cfg.py#L84)
(i.e. to ensure that just-built runtimes are used for tests, in the case
of a standalone compiler-rt build), and that is currently used by the
parent process but not the spawned ones.

My change only forwards the variable for Darwin (DYLD_LIBRARY_PATH), but
we **_ought_** to also forward the variable for other platforms.
However, it's not clear that there's any good way to plumb this into the
test, since some platforms actually have multiple library path variables
which would need to be forwarded (see: SunOS
[here](https://github.com/llvm/llvm-project/blob/main/compiler-rt/test/lit.common.cfg.py#L102)).
I considered adding a substitution variable for the library path
variable, but that doesn't really work if there's multiple such
variables.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications