Skip to content

[mlir][x86vector] Restrict BF16 dot test to x86 #124916

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
Jan 29, 2025

Conversation

adam-smnk
Copy link
Contributor

Requires x86 target for the lit test to ensure required instructions are available.

Requires x86 target for the lit test to ensure required instruction
are available.
@llvmbot
Copy link
Member

llvmbot commented Jan 29, 2025

@llvm/pr-subscribers-mlir-vector

@llvm/pr-subscribers-mlir

Author: Adam Siemieniuk (adam-smnk)

Changes

Requires x86 target for the lit test to ensure required instructions are available.


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

1 Files Affected:

  • (modified) mlir/test/Dialect/X86Vector/dot-bf16.mlir (+2)
diff --git a/mlir/test/Dialect/X86Vector/dot-bf16.mlir b/mlir/test/Dialect/X86Vector/dot-bf16.mlir
index 8c7bf6490800e3..b7ac46a66cebff 100644
--- a/mlir/test/Dialect/X86Vector/dot-bf16.mlir
+++ b/mlir/test/Dialect/X86Vector/dot-bf16.mlir
@@ -1,3 +1,5 @@
+// REQUIRES: target=x86{{.*}}
+
 // RUN: mlir-opt %s \
 // RUN:   -convert-vector-to-llvm="enable-x86vector" -convert-to-llvm \
 // RUN:   -reconcile-unrealized-casts | \

@adam-smnk
Copy link
Contributor Author

Fix for #124800

@adam-smnk adam-smnk requested a review from rengolin January 29, 2025 12:36
@adam-smnk adam-smnk merged commit 008e162 into llvm:main Jan 29, 2025
9 of 10 checks passed
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