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

[202x] Remove HLSL Effects Syntax Support #380

Open
llvm-beanz opened this issue Feb 3, 2025 · 1 comment
Open

[202x] Remove HLSL Effects Syntax Support #380

llvm-beanz opened this issue Feb 3, 2025 · 1 comment
Labels
enhancement New feature or request
Milestone

Comments

@llvm-beanz
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
DXC has never supported the "effects" runtime features, and issues warnings noting that any effects annotations are ignored. For HLSL 202x we should convert those warnings to errors and not support them in HLSL 202x.

Describe the solution you'd like
Report errors for unsupported effects syntax when used in DXC.

@llvm-beanz llvm-beanz added enhancement New feature or request needs-triage labels Feb 3, 2025
@damyanp damyanp assigned llvm-beanz and unassigned llvm-beanz Feb 3, 2025
@llvm-beanz llvm-beanz added the Design Meeting Agenda item for the design meeting label Feb 4, 2025
@llvm-beanz llvm-beanz removed the Design Meeting Agenda item for the design meeting label Feb 4, 2025
@llvm-beanz llvm-beanz added this to the HLSL 202x milestone Feb 4, 2025
@llvm-beanz
Copy link
Collaborator Author

Notes from design meeting:

@damyanp damyanp moved this to Triaged in HLSL Triage Feb 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: No status
Status: Triaged
Development

No branches or pull requests

1 participant