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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[0.5/dx12] essential fixes to depth views and command allocation #3330

Merged
merged 4 commits into from Aug 14, 2020

Conversation

kvark
Copy link
Member

@kvark kvark commented Aug 14, 2020

Fixes #2656
Tested on vange-rs and wgpu-rs examples
With these fixes, we run wgpu-rs stuff without validation errors, and terminate successfully 馃帀

@kvark kvark requested a review from msiglreith August 14, 2020 06:08
Copy link

@monocodus monocodus bot left a comment

Choose a reason for hiding this comment

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

This is an autogenerated code review.New suggestions: 6

src/backend/dx12/src/command.rs Show resolved Hide resolved
src/backend/dx12/src/conv.rs Show resolved Hide resolved
src/backend/dx12/src/device.rs Show resolved Hide resolved
src/backend/dx12/src/device.rs Show resolved Hide resolved
src/backend/dx12/src/window.rs Show resolved Hide resolved
src/backend/dx12/src/window.rs Show resolved Hide resolved
src/backend/dx12/src/device.rs Show resolved Hide resolved
Fixes the following error:
D3D12 ERROR: ID3D12Device::CreateShaderResourceView: The format (0x2d,
D24_UNORM_S8_UINT) cannot be used with a ShaderResource view. [
STATE_CREATION ERROR gfx-rs#28: CREATESHADERRESOURCEVIEW_INVALIDFORMAT]
@kvark
Copy link
Member Author

kvark commented Aug 14, 2020

Thanks for the review!
bors r=msiglreith

bors bot added a commit that referenced this pull request Aug 14, 2020
3330: [0.5/dx12] essential fixes to depth views and command allocation r=msiglreith a=kvark

Fixes #2656
Tested on vange-rs and wgpu-rs examples
With these fixes, we run wgpu-rs stuff without validation errors, and terminate successfully 馃帀 

Co-authored-by: Dzmitry Malyshau <dmalyshau@mozilla.com>
@bors
Copy link
Contributor

bors bot commented Aug 14, 2020

Build failed:

@kvark kvark merged commit 82d3984 into gfx-rs:hal-0.5 Aug 14, 2020
@kvark kvark deleted the 0.5-dx12-depth branch August 14, 2020 14:59
bors bot added a commit that referenced this pull request Aug 14, 2020
3331: [dx12] Essential fixes to depth views and command allocation r=msiglreith a=kvark

Sibling of #3330 for master

Co-authored-by: Dzmitry Malyshau <dmalyshau@mozilla.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants