kannonier8/blender
1
0
Fork 0
forked from bartvdbraak/blender
Code Pull Requests Activity

Fix build error after recent Metal GPU commit

Browse Source 
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
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; 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) static uint component_size_get(const Type gpu_type)
{ {
switch (gpu_type) { switch (gpu_type) {