Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable D3D12 experimental feature to use coopvec #6290

Conversation

jkwak-work
Copy link
Collaborator

@jkwak-work jkwak-work commented Feb 5, 2025

Close #6279

This commit enables "D3D12-experimenta-feature" in render-test.
This is required to use CoopVec feature with dxcompiler.dll.

But it is enabled only when "-dx12-experimental" is used, because it appears that DX12 becomes unstable when the experimental feature is enabled, which causes bunch of tests randomly failing.

@jkwak-work jkwak-work added the pr: non-breaking PRs without breaking changes label Feb 5, 2025
@jkwak-work jkwak-work self-assigned this Feb 5, 2025
@jkwak-work jkwak-work requested a review from a team as a code owner February 5, 2025 08:38
@jkwak-work
Copy link
Collaborator Author

/format

@slangbot
Copy link
Contributor

slangbot commented Feb 5, 2025

🌈 Formatted, please merge the changes from this PR

@jkwak-work
Copy link
Collaborator Author

A lot of dx12 tests are failing; if not all.
I need to investigate.

@@ -13,6 +13,10 @@
#include "slang-support.h"
#include "window.h"

#if defined(_WIN32)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why is this include needed?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It is needed for the type UUID.

@jkwak-work jkwak-work force-pushed the feature/support_dxc_experimental_shader_model_6_8 branch from 1d13f0d to 2701f39 Compare February 5, 2025 23:07
@jkwak-work jkwak-work marked this pull request as draft February 5, 2025 23:14
auto-merge was automatically disabled February 5, 2025 23:14

Pull request was converted to draft

@jkwak-work jkwak-work force-pushed the feature/support_dxc_experimental_shader_model_6_8 branch from 2701f39 to 6b3dd20 Compare February 5, 2025 23:56
@jkwak-work jkwak-work force-pushed the feature/support_dxc_experimental_shader_model_6_8 branch from 6b3dd20 to 5134c90 Compare February 6, 2025 01:11
@jkwak-work jkwak-work marked this pull request as ready for review February 6, 2025 01:11
@jkwak-work
Copy link
Collaborator Author

/format

@slangbot
Copy link
Contributor

slangbot commented Feb 6, 2025

🌈 Formatted, please merge the changes from this PR

@jkwak-work jkwak-work enabled auto-merge (squash) February 6, 2025 01:16
@jkwak-work
Copy link
Collaborator Author

In order to use CoopVec feature with dxcompiler.dll, Gfx had to enable "d312 experimental feature".
Initially I tried to enable it all the time.
But I observed that things became unstable when it is enabled.
So I had to make it turned off by default and it will be enabled with a command line argument "-dx12-experimental".

@jkwak-work jkwak-work merged commit 3527f09 into shader-slang:master Feb 6, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: non-breaking PRs without breaking changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support SM 6.8 in Slang Gfx
3 participants