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
Copy Markdown
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
Copy Markdown
Member

cart commented Nov 13, 2021

bors r+

bors bot pushed a commit that referenced this pull request Nov 13, 2021
@bors
Copy link
Copy Markdown
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