Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature Request] Execution Tests for Long Vectors #7069

Open
pow2clk opened this issue Jan 14, 2025 · 0 comments
Open

[Feature Request] Execution Tests for Long Vectors #7069

pow2clk opened this issue Jan 14, 2025 · 0 comments
Assignees
Labels
enhancement Feature suggestion

Comments

@pow2clk
Copy link
Member

pow2clk commented Jan 14, 2025

Execution tests for HLSL long vectors are needed to verify that vectors over size 4 are supported on all platforms.

Is your feature request related to a problem? Please describe.

See #7066 for compiler implementation. Vectors have long been limited to four elements and many existing code paths have that expectation ingrained. Whether these vectors are represented as native vectors or scalarized, testing could reveal important flaws. Scalarized vectors might be revectorized in backends and expected to be no more than 4 elements in some contexts.

Describe the solution you'd like

Existing tests for DXIL intrinsics should be leveraged to test for long vectors. This should involve a few different sizes and will probably best be done by modifying the tests to be more length robust.

@pow2clk pow2clk added enhancement Feature suggestion needs-triage Awaiting triage labels Jan 14, 2025
@pow2clk pow2clk moved this to Planning in HLSL Support Jan 14, 2025
@davidcook-msft davidcook-msft moved this from Planning to Designing in HLSL Support Jan 14, 2025
@pow2clk pow2clk changed the title [Feature Request] Execution Tests for HLSL Long Vectors [Feature Request] Execution Tests for Long Vectors Jan 14, 2025
@damyanp damyanp removed the needs-triage Awaiting triage label Jan 15, 2025
@damyanp damyanp moved this to Triaged in HLSL Triage Jan 15, 2025
@damyanp damyanp added this to the Next+1 Release milestone Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Feature suggestion
Projects
Status: New
Status: Triaged
Development

No branches or pull requests

3 participants