Skip to content

[llvm-gsymutil] Disable test macho-gsym-merged-callsites-dsym #119957

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

Merged
merged 1 commit into from
Dec 14, 2024

Conversation

alx32
Copy link
Contributor

@alx32 alx32 commented Dec 14, 2024

The macho-gsym-merged-callsites-dsym is failing on some hosts.

Disabling for now while we come up with a fix.

@alx32 alx32 requested a review from ellishg December 14, 2024 08:23
@alx32 alx32 marked this pull request as ready for review December 14, 2024 08:23
@llvmbot
Copy link
Member

llvmbot commented Dec 14, 2024

@llvm/pr-subscribers-debuginfo

Author: None (alx32)

Changes

The macho-gsym-merged-callsites-dsym is failing on some hosts.

Disabling for now while we come up with a fix.


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

1 Files Affected:

  • (modified) llvm/test/tools/llvm-gsymutil/ARM_AArch64/macho-gsym-merged-callsites-dsym.yaml (+4)
diff --git a/llvm/test/tools/llvm-gsymutil/ARM_AArch64/macho-gsym-merged-callsites-dsym.yaml b/llvm/test/tools/llvm-gsymutil/ARM_AArch64/macho-gsym-merged-callsites-dsym.yaml
index a98d4db0a791b7..caf77467bbb938 100644
--- a/llvm/test/tools/llvm-gsymutil/ARM_AArch64/macho-gsym-merged-callsites-dsym.yaml
+++ b/llvm/test/tools/llvm-gsymutil/ARM_AArch64/macho-gsym-merged-callsites-dsym.yaml
@@ -1,3 +1,7 @@
+# FIXME: Currently disabled as it fails on some Linux hosts
+# UNSUPPORTED: true
+
+
 ## Test that reconstructs a dSYM file from YAML and generates a gsym from it. The gsym has callsite info and merged functions.
 
 # RUN: split-file %s %t

@alx32 alx32 merged commit 74fb992 into llvm:main Dec 14, 2024
9 of 12 checks passed
@alx32 alx32 deleted the 05_disable_merged_callsites branch December 18, 2024 00:01
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