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.

Sign up for GitHub

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

Jump to bottom

Document Usual Arithmetic Conversions #308

Closed
llvm-beanz opened this issue Aug 27, 2024 · 0 comments · Fixed by #311
Closed

Document Usual Arithmetic Conversions #308

llvm-beanz opened this issue Aug 27, 2024 · 0 comments · Fixed by #311
Assignees
@llvm-beanz
Milestone
HLSL Spec

Comments

@llvm-beanz
Copy link
Collaborator

Which document does this relate to?
Language

Describe the issue you see with the spec
The language specification does not document how to convert arguments for builtin operator expressions.

Additional context
See C++ Usual Arithmetic Conversions
@llvm-beanz llvm-beanz mentioned this issue Aug 27, 2024
Closed
@llvm-beanz llvm-beanz self-assigned this Aug 29, 2024
@llvm-beanz llvm-beanz modified the milestones: HLSL 202x, HLSL Spec Aug 29, 2024
llvm-beanz added a commit to llvm-beanz/hlsl-specs that referenced this issue Aug 29, 2024
@llvm-beanz
Revise Usual Arithmetic Conversions
16113ba 
There are a few discrepancies between what I originally wrote here and
what DXC does. These changes adjust to the actual behavior and more
closely align with implicit conversion sequences.

Fixes microsoft#308
@llvm-beanz llvm-beanz mentioned this issue Aug 30, 2024
Merged
@damyanp damyanp moved this to Triaged in HLSL Triage Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@llvm-beanz llvm-beanz

Labels
None yet
Projects
HLSL Triage
Archived in project
Milestone
HLSL Spec
Development

Successfully merging a pull request may close this issue.

Revise Usual Arithmetic Conversions llvm-beanz/hlsl-specs
2 participants
@damyanp @llvm-beanz