Skip to content

Conversation

@thetruestblue
Copy link
Contributor

Disable this test on x86_64h for LSan.

This test is failing with malformed object only on x86_64h.
Disabling for now.

rdar://125052424

Disable this test on x86_64h for LSan.

rdar://125052424
@llvmbot
Copy link
Member

llvmbot commented Apr 29, 2024

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

Author: None (thetruestblue)

Changes

Disable this test on x86_64h for LSan.

This test is failing with malformed object only on x86_64h.
Disabling for now.

rdar://125052424


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

1 Files Affected:

  • (modified) compiler-rt/test/sanitizer_common/TestCases/sanitizer_coverage_trace_pc_guard.cpp (+2)
diff --git a/compiler-rt/test/sanitizer_common/TestCases/sanitizer_coverage_trace_pc_guard.cpp b/compiler-rt/test/sanitizer_common/TestCases/sanitizer_coverage_trace_pc_guard.cpp
index ed817961c688cc..8e14eada6b9c45 100644
--- a/compiler-rt/test/sanitizer_common/TestCases/sanitizer_coverage_trace_pc_guard.cpp
+++ b/compiler-rt/test/sanitizer_common/TestCases/sanitizer_coverage_trace_pc_guard.cpp
@@ -2,6 +2,8 @@
 
 // REQUIRES: has_sancovcc
 // UNSUPPORTED: ubsan,i386-darwin,target={{(powerpc64|s390x|thumb).*}}
+// This test is failing for lsan on darwin on x86_64.
+// UNSUPPORTED: x86_64h && lsan
 // XFAIL: tsan
 // XFAIL: android && asan
 

// REQUIRES: has_sancovcc
// UNSUPPORTED: ubsan,i386-darwin,target={{(powerpc64|s390x|thumb).*}}
// This test is failing for lsan on darwin on x86_64.
// UNSUPPORTED: x86_64h && lsan
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would we want to constrain disablement to Apple platforms? ( && darwin)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe the lit feature x86_64h is only added on Apple platforms. So I don't think the logic would actually add anything, though it would make it more clear.

@thetruestblue thetruestblue merged commit 62d6560 into llvm:main Apr 30, 2024
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