blender
/
blender
129
398
Fork 572
Code Issues 6.2k Pull Requests 750 Projects 19 Wiki Activity

Fix build error after recent Metal GPU commit 

These definitions were in the patch but didn't make it to the commit.
This commit is contained in:
Hans Goudey 2022-09-01 17:10:05 -05:00
parent af4e62a020
commit 0a85288462
Notes: blender-bot 2023-02-14 06:05:22 +01:00 
Referenced by issue #103064, Realtime Compositor: Crash on undo after dissolve node
Referenced by issue #102238, Geometry Node: Memory leak -> EXCEPTION_ACCESS_VIOLATION -> EXCEPTION_INT_DIVIDE_BY_ZERO.
1 changed files with 23 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
source/blender/gpu/intern/gpu_vertex_format.cc
View File

@ -345,6 +345,29 @@ static void VertexFormat_pack_impl(GPUVertFormat *format, uint minimum_stride)
format->packed = true;
}
void VertexFormat_pack(GPUVertFormat *format)
{
/* Perform standard vertex packing, ensuring vertex format satisfies
* minimum stride requirements for vertex assembly. */
VertexFormat_pack_impl(format, GPU_minimum_per_vertex_stride());
}
void VertexFormat_texture_buffer_pack(GPUVertFormat *format)
{
/* Validates packing for vertex formats used with texture buffers.
* In these cases, there must only be a single vertex attribute.
* This attribute should be tightly packed without padding, to ensure
* it aligns with the backing texture data format, skipping
* minimum per-vertex stride, which mandates 4-byte alignment in Metal.
* This additional alignment padding caused smaller data types, e.g. U16,
* to mis-align. */
BLI_assert_msg(format->attr_len == 1,
"Texture buffer mode should only use a single vertex attribute.");
/* Pack vertex format without minimum stride, as this is not required by texture buffers. */
VertexFormat_pack_impl(format, 1);
}
static uint component_size_get(const Type gpu_type)
{
switch (gpu_type) {