From d4fa0a0fa63c538b0c67ec3c46a45c1e4dcf91fc Mon Sep 17 00:00:00 2001 From: Kenneth Graunke Date: Sat, 6 May 2017 16:11:03 -0700 Subject: [PATCH] mesa: Make _mesa_primitive_restart_index a static inline in the header. MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit It's now basically a single expression, so it probably makes sense to have it inlined into the callers. Suggested by Marek. Reviewed-by: Timothy Arceri Reviewed-by: Marek Olšák --- src/mesa/main/varray.c | 18 ------------------ src/mesa/main/varray.h | 17 +++++++++++++++-- 2 files changed, 15 insertions(+), 20 deletions(-) diff --git a/src/mesa/main/varray.c b/src/mesa/main/varray.c index 9497090..0eb8e62 100644 --- a/src/mesa/main/varray.c +++ b/src/mesa/main/varray.c @@ -1949,24 +1949,6 @@ _mesa_VertexAttribDivisor(GLuint index, GLuint divisor) } -unsigned -_mesa_primitive_restart_index(const struct gl_context *ctx, - unsigned index_size) -{ - /* From the OpenGL 4.3 core specification, page 302: - * "If both PRIMITIVE_RESTART and PRIMITIVE_RESTART_FIXED_INDEX are - * enabled, the index value determined by PRIMITIVE_RESTART_FIXED_INDEX - * is used." - */ - if (ctx->Array.PrimitiveRestartFixedIndex) { - /* 1 -> 0xff, 2 -> 0xffff, 4 -> 0xffffffff */ - return 0xffffffffu >> 8 * (4 - index_size); - } - - return ctx->Array.RestartIndex; -} - - /** * GL_ARB_vertex_attrib_binding */ diff --git a/src/mesa/main/varray.h b/src/mesa/main/varray.h index ac49434..2d184ad 100644 --- a/src/mesa/main/varray.h +++ b/src/mesa/main/varray.h @@ -343,9 +343,22 @@ _mesa_VertexAttribDivisor_no_error(GLuint index, GLuint divisor); extern void GLAPIENTRY _mesa_VertexAttribDivisor(GLuint index, GLuint divisor); -extern unsigned +static inline unsigned _mesa_primitive_restart_index(const struct gl_context *ctx, - unsigned index_size); + unsigned index_size) +{ + /* From the OpenGL 4.3 core specification, page 302: + * "If both PRIMITIVE_RESTART and PRIMITIVE_RESTART_FIXED_INDEX are + * enabled, the index value determined by PRIMITIVE_RESTART_FIXED_INDEX + * is used." + */ + if (ctx->Array.PrimitiveRestartFixedIndex) { + /* 1 -> 0xff, 2 -> 0xffff, 4 -> 0xffffffff */ + return 0xffffffffu >> 8 * (4 - index_size); + } + + return ctx->Array.RestartIndex; +} extern void GLAPIENTRY _mesa_BindVertexBuffer(GLuint bindingIndex, GLuint buffer, GLintptr offset, -- 2.7.4