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

wgsl: specify the input type of reflect to be vector #1912

Closed
wants to merge 1 commit into from

Conversation

jzm-intel
Copy link
Contributor

Specifies the input of reflect should be vector of float.
while GLSL said "The operands must all be a scalar or vector whose component type is floating-point.", HLSL said they must be "vector", rather than "scale or vector".

Specifies the input of reflect should be vector of float.
@jzm-intel
Copy link
Contributor Author

Please take a look, thanks! @dneto0 @kvark @kainino0x @Kangz

@ben-clayton
Copy link
Collaborator

Snap!
I filed #1914 before I saw this PR.

@jzm-intel
Copy link
Contributor Author

Snap!
I filed #1914 before I saw this PR.

Yes this PR is the same with #1914.

@ben-clayton
Copy link
Collaborator

Yes this PR is the same with #1914.

#1914 also removes the scalar overload of faceForward, and may trigger a discussion about distance and length.
I'm happy if you want to land this first, or want me to remove the reflect changes from my PR.

@jzm-intel
Copy link
Contributor Author

Closed as this PR is covered by #1914, which is already landed.

@jzm-intel jzm-intel closed this Jul 8, 2021
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.

None yet

3 participants