Skip to content

[HLSL2021] Disable operators that rely on references (#4133)#4135

Merged
llvm-beanz merged 1 commit intomicrosoft:release-1.6.2112from
llvm-beanz:cbieneman/cp-disable-oo
Dec 8, 2021
Merged

[HLSL2021] Disable operators that rely on references (#4133)#4135
llvm-beanz merged 1 commit intomicrosoft:release-1.6.2112from
llvm-beanz:cbieneman/cp-disable-oo

Conversation

@llvm-beanz
Copy link
Copy Markdown
Collaborator

  • Disable operators that rely on references

After lots of discussion, we think it is best to disable operators that
should rely on reference parameters or return types for HLSL 2021. The
main driver of this is that we believe that references will get added to
HLSL in the near future, and we'd like to make it easier to maintain
compatability between HLSL 2021 and future versions of HLSL where
references are supported.

  • Removing assignment operator test case

(cherry picked from commit ade5e47)

* Disable operators that rely on references

After lots of discussion, we think it is best to disable operators that
should rely on reference parameters or return types for HLSL 2021. The
main driver of this is that we believe that references will get added to
HLSL in the near future, and we'd like to make it easier to maintain
compatability between HLSL 2021 and future versions of HLSL where
references are supported.

* Removing assignment operator test case

(cherry picked from commit ade5e47)
@AppVeyorBot
Copy link
Copy Markdown

@llvm-beanz llvm-beanz merged commit 770ac0c into microsoft:release-1.6.2112 Dec 8, 2021
@llvm-beanz llvm-beanz deleted the cbieneman/cp-disable-oo branch December 22, 2021 00:51
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