Skip to content
This repository was archived by the owner on Jan 29, 2025. It is now read-only.

wgsl-in: Splat on compound assignments #2049

Merged
merged 1 commit into from
Sep 10, 2022

Conversation

JCapucho
Copy link
Collaborator

@JCapucho JCapucho commented Sep 7, 2022

Compound assignments on wgsl follow the same semantics as their underlying operation, this includes the splatting behavior when mixing scalar and vector operands, which was done for binary operations but not for compound assignments.

Closes #2048

@JCapucho JCapucho requested a review from jimblandy September 7, 2022 22:00
Copy link
Member

@jimblandy jimblandy left a comment

Choose a reason for hiding this comment

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

Looks great. Just want the comments cleaned up a little.

Compound assignments on wgsl follow the same semantics as their
underlying operation, this includes the splatting behavior when mixing
scalar and vector operands, which was done for binary operations but not
for compound assignments.
@JCapucho JCapucho force-pushed the wgsl-splat-compound-assignments branch from 62c2178 to 87b03b2 Compare September 10, 2022 16:29
@jimblandy jimblandy merged commit db1321c into gfx-rs:master Sep 10, 2022
@JCapucho JCapucho deleted the wgsl-splat-compound-assignments branch September 10, 2022 20:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

/= operator raises a validation error
2 participants