Skip to content

[libFuzzer] Mark libFuzzer SIGTRAP test unsupported on windows #151109

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

DanBlackwell
Copy link
Contributor

This change is based on the UNSUPPORTED mark from the existing sigusr test

UNSUPPORTED: darwin, target={{.*windows.*}}

@llvmbot
Copy link
Member

llvmbot commented Jul 29, 2025

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

Author: Dan Blackwell (DanBlackwell)

Changes

This change is based on the UNSUPPORTED mark from the existing sigusr test

UNSUPPORTED: darwin, target={{.*windows.*}}


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

1 Files Affected:

  • (modified) compiler-rt/test/fuzzer/sig-trap.test (+4)
diff --git a/compiler-rt/test/fuzzer/sig-trap.test b/compiler-rt/test/fuzzer/sig-trap.test
index ee47d2c79e0e2..30d9d47f4d81f 100644
--- a/compiler-rt/test/fuzzer/sig-trap.test
+++ b/compiler-rt/test/fuzzer/sig-trap.test
@@ -1,3 +1,7 @@
+# Check that libFuzzer handles SIGTRAP; disabled on Windows due to reliance on
+# posix only features
+UNSUPPORTED: target={{.*windows.*}}
+
 RUN: %cpp_compiler %S/SigTrapTest.cpp -o %t
 
 RUN: not %run %t            2>&1 | FileCheck %s

@qinkunbao
Copy link
Member

I don't have access to a Windows computer. Let's land this PR first to see if it fixes the issue in #149120

@qinkunbao qinkunbao merged commit ba2e49c into llvm:main Jul 29, 2025
13 checks passed
@DanBlackwell
Copy link
Contributor Author

Looks like that got it building again - thanks for the review @qinkunbao !

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.

3 participants