[Mlir-commits] [mlir] [mlir:arm:tests] Add back `REQUIRED` tag removed by #216098. (PR #218447)

[llvmlistbot at llvm.org]

llvmorg-github-actions[bot] wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-mlir

Author: Ingo Müller (ingomueller-net)

<details>
<summary>Changes</summary>

The recent #<!-- -->216098 changed which features are used to select with ARM-related tests are run. Among other things, it removed the `REQUIRED` tag of one of the LIT files, `mlir/test/Integration/Dialect/Linalg/CPU/ArmNeon/matmul-i8.mlir`. That test, however, requires the `mcr_aarch64_cmd` substitution, which isn't available always. This PR adds back the tag with the `mlir_arm_neon_tests` feature, which *does* provide the necessary substitution and which seems to be the tag typically used for ARM Neon tests.

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


1 Files Affected:

- (modified) mlir/test/Integration/Dialect/Linalg/CPU/ArmNeon/matmul-i8.mlir (+2) 


``````````diff
diff --git a/mlir/test/Integration/Dialect/Linalg/CPU/ArmNeon/matmul-i8.mlir b/mlir/test/Integration/Dialect/Linalg/CPU/ArmNeon/matmul-i8.mlir
index 1a6f315cd2da4..d824208e33741 100644
--- a/mlir/test/Integration/Dialect/Linalg/CPU/ArmNeon/matmul-i8.mlir
+++ b/mlir/test/Integration/Dialect/Linalg/CPU/ArmNeon/matmul-i8.mlir
@@ -1,3 +1,5 @@
+// REQUIRES: mlir_arm_neon_tests
+
 // DEFINE: %{compile} = mlir-opt %s \
 // DEFINE:   -transform-interpreter -test-transform-dialect-erase-schedule \
 // DEFINE:   -cse -canonicalize -convert-vector-to-scf \

``````````

</details>


https://github.com/llvm/llvm-project/pull/218447