Skip to content

[AArch64] Fix for misched-branch-targets.mir test #68424

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
Oct 6, 2023

Conversation

atrosinenko
Copy link
Contributor

Fix test failure in non-assertion builds introduced by f1b2dd2.

@llvmbot
Copy link
Member

llvmbot commented Oct 6, 2023

@llvm/pr-subscribers-backend-aarch64

Changes

Fix test failure in non-assertion builds introduced by f1b2dd2.


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

1 Files Affected:

  • (modified) llvm/test/CodeGen/AArch64/misched-branch-targets.mir (+3)
diff --git a/llvm/test/CodeGen/AArch64/misched-branch-targets.mir b/llvm/test/CodeGen/AArch64/misched-branch-targets.mir
index f32c1e964f97356..ce11b095f6176ab 100644
--- a/llvm/test/CodeGen/AArch64/misched-branch-targets.mir
+++ b/llvm/test/CodeGen/AArch64/misched-branch-targets.mir
@@ -1,6 +1,9 @@
 # RUN: llc -o - -run-pass=machine-scheduler -misched=shuffle %s | FileCheck %s
 # RUN: llc -o - -run-pass=postmisched %s | FileCheck %s
 
+; REQUIRES: asserts
+; -misched=shuffle is only available with assertions enabled
+
 # Check that instructions that are recognized as branch targets by BTI
 # are not reordered by machine instruction schedulers.
 

Copy link
Collaborator

@momchil-velikov momchil-velikov left a comment

Choose a reason for hiding this comment

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

LGTM

Fix test failure in non-assertion builds introduced by
f1b2dd2.
@atrosinenko atrosinenko force-pushed the aarch64-misched-bti-quickfix branch from 58c01f7 to 4be1360 Compare October 6, 2023 15:43
@atrosinenko atrosinenko merged commit 092ef55 into llvm:main Oct 6, 2023
@atrosinenko atrosinenko deleted the aarch64-misched-bti-quickfix branch October 6, 2023 16:00
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