Skip to content

fix for mixed STORAGE_STORE and STORAGE_LOAD#903

Merged
bors[bot] merged 1 commit intogfx-rs:masterfrom
kocsis1david:mixed-buffer-storage-use
Aug 28, 2020
Merged

fix for mixed STORAGE_STORE and STORAGE_LOAD#903
bors[bot] merged 1 commit intogfx-rs:masterfrom
kocsis1david:mixed-buffer-storage-use

Conversation

@kocsis1david
Copy link
Copy Markdown
Contributor

@kocsis1david kocsis1david commented Aug 28, 2020

Connections
None

Description
A buffer that contains dynamic data is used for both uniforms and storage read. This was a problem for wgpu and it gave a PendingTransition error.

Testing
It's a trivial fix, but I don't know if there's an example for it in wgpu-rs that can be used for testing.

It seems that the PR "Sync changes from mozilla-central" is already solving the same issue.

Copy link
Copy Markdown
Contributor

@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.

Checker summary (by rust_clippy):
The tool has found 0 warnings, 1 errors.

Copy link
Copy Markdown
Contributor

@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.

Checker summary (by rust_clippy):
The tool has found 0 warnings, 1 errors.

Copy link
Copy Markdown
Contributor

@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.

Checker summary (by rust_clippy):
The tool has found 0 warnings, 1 errors.

Copy link
Copy Markdown
Member

@kvark kvark left a comment

Choose a reason for hiding this comment

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

Nice catch!
bors r+

@kvark
Copy link
Copy Markdown
Member

kvark commented Aug 28, 2020

Also, a candidate to backporting to 0.6

@kvark kvark mentioned this pull request Aug 28, 2020
6 tasks
@bors
Copy link
Copy Markdown
Contributor

bors bot commented Aug 28, 2020

@bors bors bot merged commit bba8272 into gfx-rs:master Aug 28, 2020
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.

2 participants