Skip to content

[DebugInfo] Add 'REQUIRES: object-emission' in the test. #98141

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 2 commits into from
Jul 9, 2024

Conversation

abidh
Copy link
Contributor

@abidh abidh commented Jul 9, 2024

The test added in #96474 seems to cause buildbot failure on some system. I have added REQUIRES: object-emission in the test and also used %t instead of - for output.

@llvmbot
Copy link
Member

llvmbot commented Jul 9, 2024

@llvm/pr-subscribers-debuginfo

Author: Abid Qadeer (abidh)

Changes

The test added in #96474 seems to cause buildbot failure on some system. I am marking it x86_64-linux only to sidestep the error.


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

1 Files Affected:

  • (modified) llvm/test/DebugInfo/subrange-missing-upperBound.ll (+1)
diff --git a/llvm/test/DebugInfo/subrange-missing-upperBound.ll b/llvm/test/DebugInfo/subrange-missing-upperBound.ll
index ace5bf9554c80..894ad7d2316e9 100644
--- a/llvm/test/DebugInfo/subrange-missing-upperBound.ll
+++ b/llvm/test/DebugInfo/subrange-missing-upperBound.ll
@@ -1,3 +1,4 @@
+; REQUIRES: x86_64-linux
 ; RUN: %llc_dwarf %s -filetype=obj -o - | llvm-dwarfdump - | FileCheck %s
 
 ; ModuleID = 'test.ll'

@abidh abidh changed the title [DebugInfo] Mark test to be 'x86_64-linux' only. [DebugInfo] Use %t for object file instead of -. Jul 9, 2024
@abidh abidh requested review from jmorse and dwblaikie July 9, 2024 10:30
@abidh abidh changed the title [DebugInfo] Use %t for object file instead of -. [DebugInfo] Add 'REQUIRES: object-emission' in the test. Jul 9, 2024
@dwblaikie dwblaikie merged commit 1f238ea into llvm:main Jul 9, 2024
7 checks passed
@abidh abidh deleted the subrange branch July 11, 2024 15:52
aaryanshukla pushed a commit to aaryanshukla/llvm-project that referenced this pull request Jul 14, 2024
The test added in llvm#96474 seems to cause buildbot failure on some system.
I have added `REQUIRES: object-emission` in the test and also used `%t`
instead of `-` for output.
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