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

test-server should use d3d12core.dll from bin directory #6095

Conversation

jkwak-work
Copy link
Collaborator

This commit explicitly specifies where the d3d12core.dll is located.

Currently slang-test.exe uses build/XX/bin/D3D12/d3d12core.dll whereas test-server.exe uses one from c:/windows/system32/d3d12core.dll

This commit explicitly specifies where the d3d12core.dll is located.

Currently slang-test.exe uses build/XX/bin/D3D12/d3d12core.dll whereas
test-server.exe uses one from c:/windows/system32/d3d12core.dll
@jkwak-work jkwak-work added the pr: non-breaking PRs without breaking changes label Jan 15, 2025
@jkwak-work jkwak-work self-assigned this Jan 15, 2025
@jkwak-work jkwak-work requested a review from a team as a code owner January 15, 2025 15:56
@jkwak-work
Copy link
Collaborator Author

Investigating why two tests are failing,

gfx-unit-test-tool/RayTracingTestAD3D12.internal
gfx-unit-test-tool/RayTracingTestBD3D12.internal

@jkwak-work
Copy link
Collaborator Author

The failing tests revealed a bug in Slang side.
I created an issue for it.
#6104

@jkwak-work
Copy link
Collaborator Author

I think it makes sense to mark the failing tests as expected to fail and track it in the issue 6104.

gfx-unit-test-tool/RayTracingTestAD3D12.internal
gfx-unit-test-tool/RayTracingTestBD3D12.internal

@jkwak-work
Copy link
Collaborator Author

Currently this PR is failing on a test,

FAILED test: 'tests/hlsl-intrinsic/texture/partial-resident-texture.slang (dx12)' 

@csyonghe csyonghe merged commit 87a0816 into shader-slang:master Jan 18, 2025
16 checks passed
@jkwak-work jkwak-work deleted the fix/test_server_should_use_a_custom_d3d12core_dll branch January 18, 2025 07:19
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.

2 participants