Skip to content

Conversation

Firestar99
Copy link
Member

@Firestar99 Firestar99 commented Sep 17, 2025

Instead of everyone having to implement their own Matrix4x3, we should just provide one with our #[spirv(matrix)] marker. Even our own tests have to implement their own:

#[derive(Clone, Copy)]
#[spirv(matrix)]
pub struct Matrix4x3 {
pub x: glam::Vec3,
pub y: glam::Vec3,
pub z: glam::Vec3,
pub w: glam::Vec3,
}

Required for #392, as this removes OpTypeVector from affected intrinsics.

@Firestar99 Firestar99 added this pull request to the merge queue Sep 17, 2025
Merged via the queue into main with commit fb42c2b Sep 17, 2025
13 checks passed
@Firestar99 Firestar99 deleted the matrix4x3 branch September 17, 2025 12:43
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.

2 participants