Skip to content

Mesh node attribute support#6469

Merged
pow2clk merged 10 commits intomicrosoft:release-preview-mesh-nodesfrom
pow2clk:mesh-nodes-attribs
Mar 29, 2024
Merged

Mesh node attribute support#6469
pow2clk merged 10 commits intomicrosoft:release-preview-mesh-nodesfrom
pow2clk:mesh-nodes-attribs

Conversation

@pow2clk
Copy link
Copy Markdown
Collaborator

@pow2clk pow2clk commented Mar 28, 2024

Adds new mesh node attributes NodeMaxInputRecordsPerGraphEntryRecord and OutputTopology including metadata and RDAT generation. Enhances error checking for numthreads consistent
with mesh node limits. Adds tests for all mesh node attributes and verifies errors for invalid mesh node attribute usage.

Adds minimal support for SV_DispatchGrid annotated struct params in order to enable testing NodeMaxDispatchGrid.

Fixes #6443

@pow2clk pow2clk requested a review from a team as a code owner March 28, 2024 08:47
@pow2clk pow2clk linked an issue Mar 28, 2024 that may be closed by this pull request
@pow2clk pow2clk force-pushed the mesh-nodes-attribs branch from 6d2e091 to 8f7048f Compare March 28, 2024 10:23
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 28, 2024

✅ With the latest revision this PR passed the C/C++ code formatter.

Adds new mesh node attributes NodeMaxInputRecordsPerGraphEntryRecord
and OutputTopology including metadata and RDAT generation.
Enhances error checking for numthreads consistent
with mesh node limits. Adds tests for all mesh node attributes and
verifies errors for invalid mesh node attribute usage.

Adds minimal support for SV_DispatchGrid annotated struct params in
order to enable testing NodeMaxDispatchGrid.

Fixes microsoft#6443
@pow2clk pow2clk force-pushed the mesh-nodes-attribs branch from 8f7048f to 4143726 Compare March 28, 2024 10:24
Comment thread tools/clang/test/CodeGenDXIL/hlsl/objects/NodeObjects/mesh-node-attribs.hlsl Outdated
Copy link
Copy Markdown
Contributor

@python3kgae python3kgae left a comment

Choose a reason for hiding this comment

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

Could use REQUIRES: dxil-1-9 instead of %if dxil-1-9.

Comment thread include/dxc/DXIL/DxilFunctionProps.h Outdated
Greg Roth and others added 2 commits March 28, 2024 19:44
@pow2clk pow2clk enabled auto-merge (squash) March 29, 2024 03:12
@pow2clk pow2clk merged commit 9ec9169 into microsoft:release-preview-mesh-nodes Mar 29, 2024
@pow2clk pow2clk deleted the mesh-nodes-attribs branch March 29, 2024 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

[Feature Request] Add mesh node function attributes

3 participants