Skip to content

Commit bc931a5

Browse files
authored
[mlir][spirv] Update spirv validation for module.mlir (#158678)
This is to check that the new `spirv-val` behaviour works correctly. Subsequent PRs will add validation to other target tests. This requires the most recent SPIRV-Tools to work; for existing build directories using bundled SPIR-V tools, running `SPIRVTools-update` cmake target should update them accordingly. The minimum required SPIRV-Tools version is commit `14d42009a9435c34ea55b26cc8d92240c428a052`, merged in KhronosGroup/SPIRV-Tools#6292
1 parent e6526cf commit bc931a5

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

mlir/test/Target/SPIRV/module.mlir

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
// RUN: %if spirv-tools %{ rm -rf %t %}
55
// RUN: %if spirv-tools %{ mkdir %t %}
66
// RUN: %if spirv-tools %{ mlir-translate --no-implicit-module --serialize-spirv --split-input-file --spirv-save-validation-files-with-prefix=%t/module %s %}
7-
// RUN: %if spirv-tools %{ ls %t/module*.spv | xargs -I{} spirv-val {} %}
7+
// RUN: %if spirv-tools %{ spirv-val %t %}
88

99
// CHECK: spirv.module Logical GLSL450 requires #spirv.vce<v1.0, [Shader], []> {
1010
// CHECK-NEXT: spirv.func @foo() "Inline" {

0 commit comments

Comments
 (0)