Skip to content

[SM6.10][HLK][LinAlg] Address lingering feedback#8343

Merged
V-FEXrt merged 5 commits intomicrosoft:mainfrom
V-FEXrt:lhi3
Apr 8, 2026
Merged

[SM6.10][HLK][LinAlg] Address lingering feedback#8343
V-FEXrt merged 5 commits intomicrosoft:mainfrom
V-FEXrt:lhi3

Conversation

@V-FEXrt
Copy link
Copy Markdown
Collaborator

@V-FEXrt V-FEXrt commented Apr 6, 2026

Fixes #8332

Resolve lingering feedback from the previous infrastructure PR

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 6, 2026

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

Comment thread tools/clang/unittests/HLSLExec/LinAlgTests.cpp Outdated
V-FEXrt and others added 4 commits April 6, 2026 18:46
This move requires the following non-move changes
- includes were updated
- doValuesMatch were marked inline to avoid redef errors
- using statements were added to LongVectors.cpp to match previous names
@V-FEXrt V-FEXrt marked this pull request as ready for review April 7, 2026 17:40
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.

@V-FEXrt V-FEXrt merged commit ec6ba97 into microsoft:main Apr 8, 2026
13 checks passed
@github-project-automation github-project-automation Bot moved this from New to Done in HLSL Roadmap Apr 8, 2026
@V-FEXrt V-FEXrt deleted the lhi3 branch April 8, 2026 19:14
V-FEXrt added a commit that referenced this pull request Apr 8, 2026
Build on top of #8343, will move out of draft after that merges

Internal feedback was provided to use the templated Load/Store methods.
This PR makes that change, and then simplifies the shaders/built in
defines in response to those changes.

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Basic LinAlg HLK - Framework fixes

3 participants