Skip to content

[SPIR-V] Add fspv-enable-maximal-reconvergence opt#6248

Merged
s-perron merged 4 commits into
microsoft:mainfrom
Keenuts:add-reconvergence-opt
Feb 8, 2024
Merged

[SPIR-V] Add fspv-enable-maximal-reconvergence opt#6248
s-perron merged 4 commits into
microsoft:mainfrom
Keenuts:add-reconvergence-opt

Conversation

@Keenuts
Copy link
Copy Markdown
Collaborator

@Keenuts Keenuts commented Feb 6, 2024

This option is not enabled by default. When enabled, this option adds the MaximallyReconvergesKHR execution mode to the module, and the associated extension.

Related to #6222

This option is not enabled by default. When enabled, this option
adds the MaximallyReconvergesKHR execution mode to the module, and
the associated extension.

Signed-off-by: Nathan Gauër <brioche@google.com>
Comment thread tools/clang/lib/SPIRV/CapabilityVisitor.cpp
Comment thread tools/clang/test/CodeGenSPIRV/spv.enable-maximal-reconvergence.hlsl Outdated
Comment thread tools/clang/lib/SPIRV/CapabilityVisitor.cpp
@Keenuts Keenuts requested a review from s-perron February 7, 2024 08:43
Comment thread tools/clang/test/CodeGenSPIRV/spv.enable-maximal-reconvergence.hlsl
@s-perron s-perron merged commit 9c82723 into microsoft:main Feb 8, 2024
sudonatalie added a commit to sudonatalie/DirectXShaderCompiler that referenced this pull request Mar 19, 2024
The -denorm option allows the shader to select the desired behavior
with respect to denormal values.

Note that this can't take advantage of the SPIRV-Tools capability
trimming pass for the same reasons as described in: microsoft#6248 (comment)

Fixes microsoft#6434
Keenuts pushed a commit that referenced this pull request Mar 25, 2024
The -denorm option allows the shader to select the desired behavior with
respect to denormal values.

Note that this can't take advantage of the SPIRV-Tools capability
trimming pass for the same reasons as described in:
#6248 (comment)

Fixes #6434
@Keenuts Keenuts deleted the add-reconvergence-opt branch March 25, 2024 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants