Skip to content

Remove unneeded atomicstest dependency#3631

Merged
tex3d merged 1 commit intomicrosoft:masterfrom
pow2clk:exec_atomics32
Mar 26, 2021
Merged

Remove unneeded atomicstest dependency#3631
tex3d merged 1 commit intomicrosoft:masterfrom
pow2clk:exec_atomics32

Conversation

@pow2clk
Copy link
Copy Markdown
Collaborator

@pow2clk pow2clk commented Mar 26, 2021

The 32-bit atomics test variant was requiring a UAV type unnecessarily.
By using structured buffers for groupshared output, this dependency is
removed.

The 32-bit atomics test variant was requiring a UAV type unnecessarily.
By using structured buffers for groupshared output, this dependency is
removed.
@pow2clk pow2clk requested review from hekota and jholewinski March 26, 2021 14:20
@AppVeyorBot
Copy link
Copy Markdown

Copy link
Copy Markdown
Collaborator

@jholewinski jholewinski left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

Copy link
Copy Markdown
Contributor

@tex3d tex3d left a comment

Choose a reason for hiding this comment

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

LGTM!

@tex3d tex3d merged commit 73a6706 into microsoft:master Mar 26, 2021
tex3d pushed a commit to tex3d/DirectXShaderCompiler that referenced this pull request Mar 26, 2021
The 32-bit atomics test variant was requiring a UAV type unnecessarily.
By using structured buffers for groupshared output, this dependency is
removed.

(cherry picked from commit 73a6706)
tex3d added a commit that referenced this pull request Mar 26, 2021
The 32-bit atomics test variant was requiring a UAV type unnecessarily.
By using structured buffers for groupshared output, this dependency is
removed.

(cherry picked from commit 73a6706)

Co-authored-by: Greg Roth <grroth@microsoft.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.

4 participants