Skip to content

Conversation

@wrotki
Copy link
Contributor

@wrotki wrotki commented Aug 19, 2025

It's a followup to #154189 ,
which broke test on android bot. Making sure XFAIL only happen on darwin
bots

rdar://158543555

It's a followup to llvm#154189 ,
which broke test on android bot. Making sure XFAIL only happen on darwin
bots

rdar://158543555
@llvmbot
Copy link
Member

llvmbot commented Aug 19, 2025

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

Author: Mariusz Borsa (wrotki)

Changes

It's a followup to #154189 ,
which broke test on android bot. Making sure XFAIL only happen on darwin
bots

rdar://158543555


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

2 Files Affected:

  • (modified) compiler-rt/test/asan/TestCases/Darwin/suppressions-sandbox.cpp (+1-1)
  • (modified) compiler-rt/test/asan/TestCases/Posix/fread_fwrite.cpp (+1-1)
diff --git a/compiler-rt/test/asan/TestCases/Darwin/suppressions-sandbox.cpp b/compiler-rt/test/asan/TestCases/Darwin/suppressions-sandbox.cpp
index fc1da56a969df..f12e2b2ada50d 100644
--- a/compiler-rt/test/asan/TestCases/Darwin/suppressions-sandbox.cpp
+++ b/compiler-rt/test/asan/TestCases/Darwin/suppressions-sandbox.cpp
@@ -16,7 +16,7 @@
 // UNSUPPORTED: ios
 
 // Symbolizer fails to find test functions on current macOS bot version
-// XFAIL: target=arm{{.*}}
+// XFAIL: system-darwin && target=arm{{.*}}
 
 #include <CoreFoundation/CoreFoundation.h>
 
diff --git a/compiler-rt/test/asan/TestCases/Posix/fread_fwrite.cpp b/compiler-rt/test/asan/TestCases/Posix/fread_fwrite.cpp
index 5f0be27a36318..c7b9280ea7d8e 100644
--- a/compiler-rt/test/asan/TestCases/Posix/fread_fwrite.cpp
+++ b/compiler-rt/test/asan/TestCases/Posix/fread_fwrite.cpp
@@ -3,7 +3,7 @@
 // RUN: not %run %t 1 2>&1 | FileCheck %s --check-prefix=CHECK-FREAD
 
 // Symbolizer fails to find test functions on current macOS bot version
-// XFAIL: target=arm{{.*}}
+// XFAIL: system-darwin && target=arm{{.*}}
 
 #include <stdio.h>
 #include <stdlib.h>

@wrotki wrotki merged commit 0c512f7 into llvm:main Aug 19, 2025
13 checks passed
@wrotki wrotki deleted the eng/m_borsa/narrow_xfail_constraint branch August 19, 2025 01:18
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.

2 participants