[Mlir-commits] [mlir] [mlir][ArmSME] Remove XFAILs (pending buildbot update) (PR #104758)

[Benjamin Maxwell]

https://github.com/MacDue created https://github.com/llvm/llvm-project/pull/104758

None

>From 52283974f779b63ea80879b1e51b9f2e6eaba9f5 Mon Sep 17 00:00:00 2001
From: Benjamin Maxwell <benjamin.maxwell at arm.com>
Date: Mon, 19 Aug 2024 10:47:22 +0000
Subject: [PATCH] [mlir][ArmSME] Remove XFAILs (pending buildbot update)

---
 .../CPU/ArmSME/multi-tile-matmul-mixed-types.mlir      | 10 ++--------
 .../Vector/CPU/ArmSME/outerproduct-i8i8i32.mlir        | 10 ++--------
 2 files changed, 4 insertions(+), 16 deletions(-)

diff --git a/mlir/test/Integration/Dialect/Linalg/CPU/ArmSME/multi-tile-matmul-mixed-types.mlir b/mlir/test/Integration/Dialect/Linalg/CPU/ArmSME/multi-tile-matmul-mixed-types.mlir
index 8a4c8688d64748..78815a38612e96 100644
--- a/mlir/test/Integration/Dialect/Linalg/CPU/ArmSME/multi-tile-matmul-mixed-types.mlir
+++ b/mlir/test/Integration/Dialect/Linalg/CPU/ArmSME/multi-tile-matmul-mixed-types.mlir
@@ -13,14 +13,8 @@
 /// tests a mixed i8 to i32 matmul and outer product fusion which fuses 16
 /// outer products (four per tile) into four 4-way outer products.
 
-/// NOTE: QEMU gives incorrect result for SME SMOPA 4-way outer product
-/// instruction (version <= 8.2.0, latest version at time of writing), see:
-/// https://gitlab.com/qemu-project/qemu/-/issues/2083
-/// This test is expected to fail until a fixed version of QEMU can be used.
-
-/// FIXME: Remove the 'XFAIL' below once a fixed QEMU version is available
-/// (and installed on CI buildbot).
-/// XFAIL: *
+// NOTE: QEMU <= 8.2.0 gives incorrect result for SME SMOPA 4-way outer product
+// instruction see: https://gitlab.com/qemu-project/qemu/-/issues/2083.
 
 func.func @matmul_i8_to_i32(%A : tensor<?x?xi8>, %B : tensor<?x?xi8>, %C : tensor<?x?xi32>) {
   %res = linalg.matmul ins(%A, %B: tensor<?x?xi8>, tensor<?x?xi8>)
diff --git a/mlir/test/Integration/Dialect/Vector/CPU/ArmSME/outerproduct-i8i8i32.mlir b/mlir/test/Integration/Dialect/Vector/CPU/ArmSME/outerproduct-i8i8i32.mlir
index 653b0d7beb2c78..74ddbc63380b70 100644
--- a/mlir/test/Integration/Dialect/Vector/CPU/ArmSME/outerproduct-i8i8i32.mlir
+++ b/mlir/test/Integration/Dialect/Vector/CPU/ArmSME/outerproduct-i8i8i32.mlir
@@ -7,14 +7,8 @@
 
 // RUN: %{compile} | %{run} | FileCheck %s
 
-// NOTE: QEMU gives incorrect result for SME SMOPA 4-way outer product
-// instruction (version <= 8.2.0, latest version at time of writing), see:
-// https://gitlab.com/qemu-project/qemu/-/issues/2083 This test is expected to
-// fail (CHECK lines are correct) until a fixed version of QEMU can be used.
-
-// FIXME: Remove the 'XFAIL' below once a fixed QEMU version is available
-// (and installed on CI buildbot).
-// XFAIL: *
+// NOTE: QEMU <= 8.2.0 gives incorrect result for SME SMOPA 4-way outer product
+// instruction see: https://gitlab.com/qemu-project/qemu/-/issues/2083.
 
 // NOTE: there is no non-widening variant for these types and this test can't
 // be lowered without the widening pass, therefore we can't check if the result