[MLIR][test][nfc] Add asserts requirement to one test

[yozhu]

No description provided.

[llvmbot]

@llvm/pr-subscribers-mlir

@llvm/pr-subscribers-mlir-spirv

Author: YongKang Zhu (yozhu)

Changes

---

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

1 Files Affected:

• (modified) mlir/test/Target/SPIRV/function-decorations.mlir (+1)

diff --git a/mlir/test/Target/SPIRV/function-decorations.mlir b/mlir/test/Target/SPIRV/function-decorations.mlir
index 6098e42f063a2..ad59d01cccef4 100644
--- a/mlir/test/Target/SPIRV/function-decorations.mlir
+++ b/mlir/test/Target/SPIRV/function-decorations.mlir
@@ -1,3 +1,4 @@
+// REQUIRES: asserts
 // RUN: mlir-translate --no-implicit-module --test-spirv-roundtrip --split-input-file %s | FileCheck %s
 // RUN: mlir-translate --no-implicit-module --test-spirv-roundtrip --split-input-file --debug %s | FileCheck %s
 

[kuhar]

Could you explain why this is required? If we have some code behind NDEBUG, can we make it unconditional instead?

[yozhu]

Could you explain why this is required? If we have some code behind NDEBUG, can we make it unconditional instead?

When this test is run with a build that doesn't enable DEBUG, the test will fail with error message mlir-translate: Unknown command line argument '--debug'.

[kuhar]

We should probably comment out the RUN line with --debug then

[kuhar]

This was resolved in #164319