Skip to content

Add local resources tests#1090

Open
bob80905 wants to merge 8 commits intollvm:mainfrom
bob80905:local_resources_tests
Open

Add local resources tests#1090
bob80905 wants to merge 8 commits intollvm:mainfrom
bob80905:local_resources_tests

Conversation

@bob80905
Copy link
Copy Markdown
Contributor

@bob80905 bob80905 commented Apr 15, 2026

This PR adds tests for various scenarios of local resources.
Tests included here are guaranteed to produce output on either the Clang or DXC compilers.
This PR flows out of the design in llvm/wg-hlsl#414

Addresses llvm/wg-hlsl#180
Assisted by: Github Copilot

@bob80905 bob80905 added the test-all When applied to a PR this will opt-in to additional pre-merge test configurations.. label Apr 16, 2026
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What is this file for?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This was accidentally added. I believe it was used by Copilot to keep track of compilation results / diagnostics, but it should not be part of this PR. I'll remove it.

@Icohedron
Copy link
Copy Markdown
Contributor

If Copilot was involved in the making of this PR then you need to state it in your PR description at the very least.
Write "Assisted-by: GitHub Copilot"

@bob80905
Copy link
Copy Markdown
Contributor Author

If Copilot was involved in the making of this PR then you need to state it in your PR description at the very least. Write "Assisted-by: GitHub Copilot"

Missed that! Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test-all When applied to a PR this will opt-in to additional pre-merge test configurations..

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants