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

DXC produces invalid DXIL on dot using argument with double type. #6566

Closed
python3kgae opened this issue Apr 24, 2024 · 2 comments
Closed

DXC produces invalid DXIL on dot using argument with double type. #6566

python3kgae opened this issue Apr 24, 2024 · 2 comments
Labels
bug Bug, regression, crash

Comments

@python3kgae
Copy link
Contributor

python3kgae commented Apr 24, 2024

Description
dot(a, b) will generate invalid DXIL when a and b are vectors of double.
The hlsl intrinsic allows double vector while the dxil op for dot not support double.
Steps to Reproduce

dxc -Tps_6_0

double2 a;
double2 b;

float main() : SV_Target {
return dot(a, b);
}

Actual Behavior

DXC generated DXIL that failed validation.
Environment

  • DXC version : 1.8
  • Host Operating System : Windows
@tex3d
Copy link
Contributor

tex3d commented Apr 24, 2024

To be clear, the crash is a temporary regression in the main branch and original 1.8 release that was fixed in point release. The prior behavior, the current behavior of latest 1.8 releases, and the behavior for main once #6543 merges, will be to produce a validation error due to an illegal DXIL op overload instead.

@damyanp damyanp changed the title DXC crash on dot using argument with double type. DXC produces invalid DXIL on dot using argument with double type. Apr 25, 2024
@damyanp damyanp removed the needs-triage Awaiting triage label Apr 25, 2024
@damyanp
Copy link
Member

damyanp commented Apr 25, 2024

We have no plans to make changes in this area in DXC without considering a design that takes into account what the design is in Clang. For this reason I'm closing this issue as not planned.

@damyanp damyanp closed this as not planned Won't fix, can't repro, duplicate, stale Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bug, regression, crash
Projects
Archived in project
Development

No branches or pull requests

3 participants