Skip to content
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

Fix test to have correct requirements #95106

Merged
merged 1 commit into from
Jun 11, 2024
Merged

Fix test to have correct requirements #95106

merged 1 commit into from
Jun 11, 2024

Conversation

pogo59
Copy link
Collaborator

@pogo59 pogo59 commented Jun 11, 2024

No description provided.

@pogo59 pogo59 merged commit 32add24 into llvm:main Jun 11, 2024
4 of 6 checks passed
@llvmbot
Copy link
Collaborator

llvmbot commented Jun 11, 2024

@llvm/pr-subscribers-backend-arm

Author: Paul T Robinson (pogo59)

Changes

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

1 Files Affected:

  • (modified) llvm/test/CodeGen/ARM/apple-version-min.ll (+3)
diff --git a/llvm/test/CodeGen/ARM/apple-version-min.ll b/llvm/test/CodeGen/ARM/apple-version-min.ll
index 6b4af21d74c00..180c19e5e3623 100644
--- a/llvm/test/CodeGen/ARM/apple-version-min.ll
+++ b/llvm/test/CodeGen/ARM/apple-version-min.ll
@@ -1,5 +1,8 @@
 ; Test emitting version_min directives.
 
+; Let's not split this into separate ARM/AArch64 parts.
+; REQUIRES: aarch64-registered-target
+
 ; RUN: llc %s -filetype=asm -o - --mtriple arm64-apple-tvos9.0.0 | FileCheck %s --check-prefix=TVOS
 ; RUN: llc %s -filetype=asm -o - --mtriple thumbv7s-apple-ios7.0.0 | FileCheck %s --check-prefix=IOS
 ; RUN: llc %s -filetype=asm -o - --mtriple thumbv7k-apple-watchos2.0.0 | FileCheck %s --check-prefix=WATCHOS

@pogo59
Copy link
Collaborator Author

pogo59 commented Jun 11, 2024

@jroelofs If you rather I did split this into two tests, LMK

@jroelofs
Copy link
Contributor

LGTM

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