Skip to content

[SPIR-V] Please considering emitting SPV_KHR_compute_shader_derivatives extension conditionally #8886

Description

@theraininsky

Description

The previous fix for #7943 emits additional extension requirement and execution mode globally for shaders uses QuadReadAcross* intrinsics, which breaks so many legacy devices. I know It better reflects QuadReadAcross* intrinsics in spirv which is good, but I think the most of the stuff been talked about in that thread should be targeting SM6.6, at least the required vulkan extension VK_KHR_compute_shader_derivatives is targeting SM6.6. So please consider at least gating that fix for SM6.6+only, as it really should not be enabled by default on SM6.0.
At least on mobile deivces, there's few device actually supports this extension, for example samsung doesn't support this extension until S26 (Adreno 840)

Steps to Reproduce

Actual Behavior

Environment

  • DXC version
  • Host Operating System <!--- Host operating system and version --->

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugBug, regression, crashneeds-triageAwaiting triagespirvWork related to SPIR-V

    Type

    No type

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions