Skip to content

AArch64: Remove incorrect REQUIRES arm-registered-target from test#111983

Merged
arsenm merged 1 commit intomainfrom
users/arsenm/aarch64-remove-wrong-requires-registered-target
Oct 12, 2024
Merged

AArch64: Remove incorrect REQUIRES arm-registered-target from test#111983
arsenm merged 1 commit intomainfrom
users/arsenm/aarch64-remove-wrong-requires-registered-target

Conversation

@arsenm
Copy link
Copy Markdown
Contributor

@arsenm arsenm commented Oct 11, 2024

The test only depends on AArch64 as it should.

Copy link
Copy Markdown
Contributor Author

arsenm commented Oct 11, 2024

@llvmbot
Copy link
Copy Markdown
Member

llvmbot commented Oct 11, 2024

@llvm/pr-subscribers-backend-aarch64

Author: Matt Arsenault (arsenm)

Changes

The test only depends on AArch64 as it should.


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

1 Files Affected:

  • (modified) llvm/test/CodeGen/AArch64/vecreduce-propagate-sd-flags.ll (-1)
diff --git a/llvm/test/CodeGen/AArch64/vecreduce-propagate-sd-flags.ll b/llvm/test/CodeGen/AArch64/vecreduce-propagate-sd-flags.ll
index 823fdfb4809768..830f1efbdf67f6 100644
--- a/llvm/test/CodeGen/AArch64/vecreduce-propagate-sd-flags.ll
+++ b/llvm/test/CodeGen/AArch64/vecreduce-propagate-sd-flags.ll
@@ -1,4 +1,3 @@
-; REQUIRES: arm-registered-target
 ; REQUIRES: asserts
 ; RUN: llc -o /dev/null %s -debug-only=legalize-types 2>&1 | FileCheck %s
 

Base automatically changed from users/arsenm/target-tests-remove-redundant-requires to main October 11, 2024 12:16
The test only depends on AArch64 as it should.
@arsenm arsenm force-pushed the users/arsenm/aarch64-remove-wrong-requires-registered-target branch from b0146e1 to 5dab2ee Compare October 11, 2024 12:17
@arsenm arsenm merged commit cb2f161 into main Oct 12, 2024
@arsenm arsenm deleted the users/arsenm/aarch64-remove-wrong-requires-registered-target branch October 12, 2024 09:26
DanielCChen pushed a commit to DanielCChen/llvm-project that referenced this pull request Oct 16, 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