Skip to content

[Backport to 21] fixes a new validation failure in a UniformId test (…#3492

Merged
MrSidims merged 1 commit intoKhronosGroup:llvm_release_210from
MrSidims:llvm_release_210-uniid-test
Jan 10, 2026
Merged

[Backport to 21] fixes a new validation failure in a UniformId test (…#3492
MrSidims merged 1 commit intoKhronosGroup:llvm_release_210from
MrSidims:llvm_release_210-uniid-test

Conversation

@MrSidims
Copy link
Contributor

@MrSidims MrSidims commented Jan 8, 2026

#3301)

A recent spirv-val change requires that OpDecorateId IDs are well-ordered, which means that the decoration operand ID cannot be the same as the decoration target ID. See:
KhronosGroup/SPIRV-Tools#6227

This leads to the failure:

error: line 6: Parameter <ID> '2[%uint_0]' must appear earlier in the binary than the target
  OpDecorateId %uint_0 UniformId %uint_0

The fix is to use a different ID for the decoration operand and the decoration target.

…hronosGroup#3301)

A recent spirv-val change requires that OpDecorateId IDs are
well-ordered, which means that the decoration operand ID cannot be the
same as the decoration target ID. See:
KhronosGroup/SPIRV-Tools#6227

This leads to the failure:

```
error: line 6: Parameter <ID> '2[%uint_0]' must appear earlier in the binary than the target
  OpDecorateId %uint_0 UniformId %uint_0
```

The fix is to use a different ID for the decoration operand and the
decoration target.
@MrSidims MrSidims merged commit 14f03f1 into KhronosGroup:llvm_release_210 Jan 10, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants