Skip to content

[Spirv] Fix for validation error VUID-StandaloneSpirv-Flat-04744#5276

Merged
Keenuts merged 8 commits intomicrosoft:mainfrom
rajivmg:fix-VUID-StandaloneSpirv-Flat-04744
Jun 9, 2023
Merged

[Spirv] Fix for validation error VUID-StandaloneSpirv-Flat-04744#5276
Keenuts merged 8 commits intomicrosoft:mainfrom
rajivmg:fix-VUID-StandaloneSpirv-Flat-04744

Conversation

@rajivmg
Copy link
Copy Markdown
Contributor

@rajivmg rajivmg commented Jun 7, 2023

We get this validation error, if the pixel shader use WaveGetLaneIndex() or WaveGetLaneCount() functions.

generated SPIR-V is invalid: [VUID-StandaloneSpirv-Flat-04744] Fragment OpEntryPoint operand 4 with Input interfaces with integer or float type must have a Flat decoration for Entry Point id 1.
%SubgroupLocalInvocationId = OpVariable %_ptr_Input_uint Input

Cause of the error: https://github.com/KhronosGroup/SPIRV-Tools/blob/59b4febd819977b440692674f6a862d6d17731b5/source/val/validate_decorations.cpp#L876

Fixes #5014

rajivmg added 4 commits May 13, 2023 15:51
generated SPIR-V is invalid: [VUID-StandaloneSpirv-Flat-04744] Fragment OpEntryPoint operand 4 with Input interfaces with integer or float type must have a Flat decoration for Entry Point id 1.
  %SubgroupLocalInvocationId = OpVariable %_ptr_Input_uint Input
@rajivmg rajivmg changed the title [Spirv] Test for validation error VUID-StandaloneSpirv-Flat-04744 e60a520c8 [Spirv] Fix for validation error VUID-StandaloneSpirv-Flat-04744 Jun 7, 2023
@AppVeyorBot
Copy link
Copy Markdown

@AppVeyorBot
Copy link
Copy Markdown

@AppVeyorBot
Copy link
Copy Markdown

@rajivmg
Copy link
Copy Markdown
Contributor Author

rajivmg commented Jun 9, 2023

@llvm-beanz @tex3d @Keenuts @vettoreldaniele @sverrejoh @s-perron // Please review this change.. :)

@s-perron s-perron requested a review from Keenuts June 9, 2023 13:24
Copy link
Copy Markdown
Collaborator

@Keenuts Keenuts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Keenuts Keenuts enabled auto-merge (squash) June 9, 2023 14:24
@rajivmg
Copy link
Copy Markdown
Contributor Author

rajivmg commented Jun 9, 2023

All tests succeeded 2 days ago but this is still showing as not completed. If there is a way can someone please retrigger the pipeline.
pipeline

@Keenuts Keenuts merged commit a742bb6 into microsoft:main Jun 9, 2023
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.

[SPIR-V] WaveGetLaneIndex() genererates invalid SPIR-V in fragment shader

4 participants