[all-commits] [llvm/llvm-project] 81cd73: [compiler-rt][test] Rewrote test to fix usage of e...

[Connie Zhu via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 81cd7358a2ce411c99512d055e98d0b9dc119c14
      https://github.com/llvm/llvm-project/commit/81cd7358a2ce411c99512d055e98d0b9dc119c14
  Author: Connie Zhu <60797237+connieyzhu at users.noreply.github.com>
  Date:   2024-08-27 (Tue, 27 Aug 2024)

  Changed paths:
    M compiler-rt/test/asan/TestCases/Posix/high-address-dereference.c

  Log Message:
  -----------
  [compiler-rt][test] Rewrote test to fix usage of export with 2 arguments (#106143)

This patch fixes the incorrect usage of lit's built-in export command in
this compiler-rt test. The export command only allows for one argument,
but %env_asan_opts expands to "env ASAN_OPTIONS", which gives export two
arguments instead of only one. Instead of using export, which propogates
the environment variable value to all subsequent commands, the
environment variable is manually set before all RUN lines following the
original export line.

This patch depends on https://github.com/llvm/llvm-project/pull/105961.

Fixes https://github.com/llvm/llvm-project/issues/106142.



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