From 06588a065fdfd642917af5710fbedcbc941bb97d Mon Sep 17 00:00:00 2001 From: Brian Paul Date: Fri, 15 Dec 2017 15:08:17 -0700 Subject: [PATCH] glsl: document varying_matches::assign_locations() params and return value And change *components to components[] as a reminder that it's an array. Reviewed-by: Timothy Arceri --- src/compiler/glsl/link_varyings.cpp | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/src/compiler/glsl/link_varyings.cpp b/src/compiler/glsl/link_varyings.cpp index a1c2d424d40..7f42f8a2150 100644 --- a/src/compiler/glsl/link_varyings.cpp +++ b/src/compiler/glsl/link_varyings.cpp @@ -1468,7 +1468,7 @@ public: ~varying_matches(); void record(ir_variable *producer_var, ir_variable *consumer_var); unsigned assign_locations(struct gl_shader_program *prog, - uint8_t *components, + uint8_t components[], uint64_t reserved_slots); void store_locations() const; @@ -1742,10 +1742,15 @@ varying_matches::record(ir_variable *producer_var, ir_variable *consumer_var) /** * Choose locations for all of the variable matches that were previously * passed to varying_matches::record(). + * \param components returns array[slot] of number of components used + * per slot (1, 2, 3 or 4) + * \param reserved_slots bitmask indicating which varying slots are already + * allocated + * \return number of slots (4-element vectors) allocated */ unsigned varying_matches::assign_locations(struct gl_shader_program *prog, - uint8_t *components, + uint8_t components[], uint64_t reserved_slots) { /* If packing has been disabled then we cannot safely sort the varyings by -- 2.30.2