Skip to content

Conversation

@cmtice
Copy link
Contributor

@cmtice cmtice commented Sep 10, 2025

Update sig-trap.test to work with the lit internal shell, as part of our migration to make the internal shell the default for lit tests.

Update sig-trap.test to work with the lit internal shell, as part
of our migration to make the internal shell the default for lit
tests.
@llvmbot
Copy link
Member

llvmbot commented Sep 10, 2025

@llvm/pr-subscribers-compiler-rt-sanitizer

Author: None (cmtice)

Changes

Update sig-trap.test to work with the lit internal shell, as part of our migration to make the internal shell the default for lit tests.


Full diff: https://github.com/llvm/llvm-project/pull/157966.diff

1 Files Affected:

  • (modified) compiler-rt/test/fuzzer/sig-trap.test (+1-1)
diff --git a/compiler-rt/test/fuzzer/sig-trap.test b/compiler-rt/test/fuzzer/sig-trap.test
index 60208c486a87a..1672d1e0229c7 100644
--- a/compiler-rt/test/fuzzer/sig-trap.test
+++ b/compiler-rt/test/fuzzer/sig-trap.test
@@ -8,4 +8,4 @@ RUN: not %run %t            2>&1 | FileCheck %s
 CHECK-DAG: BINGO
 CHECK-DAG: ERROR: libFuzzer: deadly signal
 
-RUN: trap "%run %t -handle_trap=0" TRAP
+RUN: bash -c "trap '%run %t -handle_trap=0' TRAP"

@cmtice cmtice merged commit afaea7f into llvm:main Sep 11, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants