Skip to content

[Merged by Bors] - replace matrix swizzles in pbr shader with index accesses#3122

Closed
jakobhellermann wants to merge 1 commit intobevyengine:pipelined-renderingfrom
jakobhellermann:fix-pbr-shader-matrix-swizzles
Closed

[Merged by Bors] - replace matrix swizzles in pbr shader with index accesses#3122
jakobhellermann wants to merge 1 commit intobevyengine:pipelined-renderingfrom
jakobhellermann:fix-pbr-shader-matrix-swizzles

Conversation

@jakobhellermann
Copy link
Contributor

Matrix swizzles like mat.x.xyz are not supported in WGSL and accepted in naga by accident: https://gpuweb.github.io/gpuweb/wgsl/#matrix-access-expr

@jakobhellermann jakobhellermann added A-Rendering Drawing game state to the screen S-Needs-Review labels Nov 13, 2021
@cart
Copy link
Member

cart commented Nov 13, 2021

bors r+

bors bot pushed a commit that referenced this pull request Nov 13, 2021
Matrix swizzles like `mat.x.xyz` are not supported in WGSL and accepted in naga by accident: <https://gpuweb.github.io/gpuweb/wgsl/#matrix-access-expr>
@bors
Copy link
Contributor

bors bot commented Nov 13, 2021

@bors bors bot changed the title replace matrix swizzles in pbr shader with index accesses [Merged by Bors] - replace matrix swizzles in pbr shader with index accesses Nov 13, 2021
@bors bors bot closed this Nov 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Rendering Drawing game state to the screen

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants