Update D3D LinAlg matrix construction caps query shape

[Jesse Natalie (jenatali)]

Implement the changes in microsoft/DirectX-Specs#257

[Copilot]

Copilot review overview

🟡 Changes recommended

Thread-group paths query incorrect scope or “any” wave size, potentially running unsupported configurations.

Review effort: Balanced
Findings: 2

Open (2)

• Use candidate wave size for thread-group capability queries · New
• Use thread-group scope and size in multiply selector queries · New

What changed in this PR

Updates LinAlg execution tests to match the revised D3D12 matrix-construction capability API.

Changes:

• Adds matrix use, scope, thread-group size, and M×N construction shapes.
• Updates capability query marshalling and logging.
• Queries construction support per matrix role.

File	Description
LinAlgTests.cpp	Updates capability gating and matrix-role queries.
HlslExecTestUtils.h	Mirrors the revised runtime ABI.
HlslExecTestUtils.cpp	Marshals and logs the new query fields.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Copilot]

Copilot review overview

🔵 Needs a closer look

The hardware-specific runtime ABI changes require final validation against the preview host API and supported devices.

Review effort: Balanced
Findings: None

Resolved since last review (2)

• Use thread-group scope and size in multiply selector queries
• Use candidate wave size for thread-group capability queries

Previously missed (2)

In code that hasn't changed since last review

Update comment to describe exact construction query semantics

tools/​clang/​unittests/​HLSLExec/​HlslExecTestUtils.h:518

The comment above this struct still says the runtime applies the “positive multiple of a native tile” rule. The v0.10 specification removes that guarantee and defines support for the exact component type/use/scope/group-size/shape tuple, so this wording can lead callers to assume unsupported closure under tile multiples. Please update it to describe the exact construction query instead.

Clarify role-dependent construction support and fixed query shape

tools/​clang/​unittests/​HLSLExec/​LinAlgTests.cpp:312

The Uses explanation above this loop still says each role pins different extents of one {M,K,N} shape. The new API sends the same exact Params.M × Params.N shape for each iteration and varies Use; the reason all roles must pass is now role-dependent construction support. Please update that explanation (and the similar MatAccum wording) so it matches the new query semantics.

[Copilot]

Copilot review overview

🟢 Approval recommended

The changes consistently implement the merged specification’s revised matrix-construction query contract.

Review effort: Balanced
Findings: None