Skip to content

Conversation

llvm-beanz
Copy link
Collaborator

The -Gis flag opts into "strict" IEEE math rules, which include preserving NaN, Inf and signed/unsigned zeros. This change adds the SPIRV SignedZeroInfNanPreserve capability and execution mode when -Gis is passed to DXC.

The `-Gis` flag opts into "strict" IEEE math rules, which include
preserving NaN, Inf and signed/unsigned zeros. This change adds the
SPIRV SignedZeroInfNanPreserve capability and execution mode when `-Gis`
is passed to DXC.

../tools/clang/test/CodeGenSPIRV/SignedZeroInfNanPreserve.hlsl
Copy link
Collaborator

@bob80905 bob80905 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
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.

Thanks! Just the extension possibility, otherwise if there is a reason not to, looks good

../tools/clang/test/CodeGenSPIRV/SignedZeroInfNanPreserve.hlsl
Copy link
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.

Minor nit on the CHECK line name, otherwise LGTM,thanks!

@llvm-beanz llvm-beanz merged commit 7a8c90c into microsoft:main Aug 18, 2025
12 checks passed
@github-project-automation github-project-automation bot moved this from New to Done in HLSL Roadmap Aug 18, 2025
@llvm-beanz llvm-beanz deleted the cbieneman/SignedZeroInfNanPreserve branch August 18, 2025 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants