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

[Overload.Res] Add overload resolution sections #244

Merged
merged 2 commits into from
Jun 11, 2024

Conversation

llvm-beanz
Copy link
Collaborator

This starts to flesh out more complete overload resolution rules. A lot of this is extremely similar to C++ however we have some key differences. Those differences include:

  • HLSL supports overload resolution in 5 contexts, whereas C++ supports it in 7 (DXC actually only supports 2 of those contexts).
  • HLSL has two additional conversion sequence ranks.
  • HLSL has more rules to disambiguate some conversions.

This starts to flesh out more complete overload resolution rules. A lot
of this is extremely similar to C++ however we have some key
differences. Those differences include:

* HLSL supports overload resolution in 5 contexts, whereas C++ supports
  it in 7 (DXC actually only supports 2 of those contexts).
* HLSL has two additional conversion sequence ranks.
* HLSL has more rules to disambiguate some conversions.
Copy link
Member

@hekota hekota left a comment

Choose a reason for hiding this comment

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

LGTM!

@llvm-beanz llvm-beanz merged commit 69c08a5 into microsoft:main Jun 11, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants