[Mlir-commits] [mlir] [mlir][test] XFAIL little-endian-only tests on SPARC (PR #103726)
llvmlistbot at llvm.org
llvmlistbot at llvm.org
Wed Aug 14 02:19:50 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-mlir-spirv
Author: Rainer Orth (rorth)
<details>
<summary>Changes</summary>
3 MLIR tests `FAIL` on SPARC, both Solaris/sparcv9 and Linux/sparc64:
```
MLIR :: Conversion/ArithToSPIRV/arith-to-spirv-le-specific.mlir
MLIR :: IR/elements-attr-interface.mlir
MLIR :: Target/LLVMIR/llvmir-le-specific.mlir
```
The issue is always the same: the tests in question are little-endian-only currently, so this patch `XFAIL`s them on `sparc*` as is already done for `s390x`.
Tested on `sparcv9-sun-solaris2.11`, `sparc64-unknown-linux-gnu`, `amd64-pc-solaris2.11`, and `x86_64-pc-linux-gnu`.
---
Full diff: https://github.com/llvm/llvm-project/pull/103726.diff
3 Files Affected:
- (modified) mlir/test/Conversion/ArithToSPIRV/arith-to-spirv-le-specific.mlir (+1-1)
- (modified) mlir/test/IR/elements-attr-interface.mlir (+1-1)
- (modified) mlir/test/Target/LLVMIR/llvmir-le-specific.mlir (+2-2)
``````````diff
diff --git a/mlir/test/Conversion/ArithToSPIRV/arith-to-spirv-le-specific.mlir b/mlir/test/Conversion/ArithToSPIRV/arith-to-spirv-le-specific.mlir
index 7233a8bfffa9db..47be1be30577d8 100644
--- a/mlir/test/Conversion/ArithToSPIRV/arith-to-spirv-le-specific.mlir
+++ b/mlir/test/Conversion/ArithToSPIRV/arith-to-spirv-le-specific.mlir
@@ -10,7 +10,7 @@
//
//===----------------------------------------------------------------------===//
-// XFAIL: target=s390x-{{.*}}
+// XFAIL: target={{(s390x|sparc.*)-.*}}
module attributes {
spirv.target_env = #spirv.target_env<
diff --git a/mlir/test/IR/elements-attr-interface.mlir b/mlir/test/IR/elements-attr-interface.mlir
index 5234c81bd841e3..79283f1aae99a8 100644
--- a/mlir/test/IR/elements-attr-interface.mlir
+++ b/mlir/test/IR/elements-attr-interface.mlir
@@ -1,7 +1,7 @@
// RUN: mlir-opt %s -test-elements-attr-interface -verify-diagnostics
// Parsing external resources does not work on big-endian platforms currently
-// XFAIL: target=s390x-{{.*}}
+// XFAIL: target={{(s390x|sparc.*)-.*}}
// This test contains various `ElementsAttr` attributes, and tests the support
// for iterating the values of these attributes using various native C++ types.
diff --git a/mlir/test/Target/LLVMIR/llvmir-le-specific.mlir b/mlir/test/Target/LLVMIR/llvmir-le-specific.mlir
index f8d082082117cb..98145bc35cba77 100644
--- a/mlir/test/Target/LLVMIR/llvmir-le-specific.mlir
+++ b/mlir/test/Target/LLVMIR/llvmir-le-specific.mlir
@@ -1,7 +1,7 @@
// RUN: mlir-translate -mlir-to-llvmir -split-input-file %s | FileCheck %s
// Decoding the attribute does not work on big-endian platforms currently
-// XFAIL: target=s390x-{{.*}}
+// XFAIL: target={{(s390x|sparc.*)-.*}}
// CHECK{LITERAL}: @dense_resource_tensor_constant = internal constant [5 x float] [float 0x3FCA034080000000, float 0xBFD0466300000000, float 0xBFD75DDF80000000, float 0xBFDE074F40000000, float 0x3FDDD3A1C0000000]
llvm.mlir.global internal constant @dense_resource_tensor_constant(dense_resource<dense_resource_test_5xf32> : tensor<5xf32>) : !llvm.array<5 x f32>
@@ -24,4 +24,4 @@ llvm.mlir.global internal constant @dense_resource_multidim_vector_constant(dens
dense_resource_test_2x2xf32: "0x0800000054A3B53ED6C0B33E55D1A2BDE5D2BB3E"
}
}
-#-}
\ No newline at end of file
+#-}
``````````
</details>
https://github.com/llvm/llvm-project/pull/103726
More information about the Mlir-commits
mailing list