Mesa (master): gallium/nir/tgsi: reindent some code in the nir->tgsi info (v2)

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Wed Aug 19 01:23:15 UTC 2020


Module: Mesa
Branch: master
Commit: 407fa329a4b576d5747662f2caeb752cb44ade6d
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=407fa329a4b576d5747662f2caeb752cb44ade6d

Author: Dave Airlie <airlied at redhat.com>
Date:   Wed Aug 19 04:52:06 2020 +1000

gallium/nir/tgsi: reindent some code in the nir->tgsi info (v2)

v2: Eric pointed out some code changes snuck in here.

Reviewed-by: Roland Scheidegger <sroland at vmware.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6339>

---

 src/gallium/auxiliary/nir/nir_to_tgsi_info.c | 130 +++++++++++++--------------
 1 file changed, 65 insertions(+), 65 deletions(-)

diff --git a/src/gallium/auxiliary/nir/nir_to_tgsi_info.c b/src/gallium/auxiliary/nir/nir_to_tgsi_info.c
index 25b7066bc6e..d03dbd39b0d 100644
--- a/src/gallium/auxiliary/nir/nir_to_tgsi_info.c
+++ b/src/gallium/auxiliary/nir/nir_to_tgsi_info.c
@@ -55,80 +55,80 @@ static nir_variable* intrinsic_get_var(nir_intrinsic_instr *instr)
 
 
 static void gather_usage_helper(const nir_deref_instr **deref_ptr,
-				unsigned location,
-				uint8_t mask,
-				uint8_t *usage_mask)
+                                unsigned location,
+                                uint8_t mask,
+                                uint8_t *usage_mask)
 {
-	for (; *deref_ptr; deref_ptr++) {
-		const nir_deref_instr *deref = *deref_ptr;
-		switch (deref->deref_type) {
-		case nir_deref_type_array: {
-			unsigned elem_size =
-				glsl_count_attribute_slots(deref->type, false);
-			if (nir_src_is_const(deref->arr.index)) {
-				location += elem_size * nir_src_as_uint(deref->arr.index);
-			} else {
-				unsigned array_elems =
-					glsl_get_length(deref_ptr[-1]->type);
-				for (unsigned i = 0; i < array_elems; i++) {
-					gather_usage_helper(deref_ptr + 1,
-							    location + elem_size * i,
-							    mask, usage_mask);
-				}
-				return;
-			}
-			break;
-		}
-		case nir_deref_type_struct: {
-			const struct glsl_type *parent_type =
-				deref_ptr[-1]->type;
-			unsigned index = deref->strct.index;
-			for (unsigned i = 0; i < index; i++) {
-				const struct glsl_type *ft = glsl_get_struct_field(parent_type, i);
-				location += glsl_count_attribute_slots(ft, false);
-			}
-			break;
-		}
-		default:
-			unreachable("Unhandled deref type in gather_components_used_helper");
-		}
-	}
-
-	usage_mask[location] |= mask & 0xf;
-	if (mask & 0xf0)
-		usage_mask[location + 1] |= (mask >> 4) & 0xf;
+   for (; *deref_ptr; deref_ptr++) {
+      const nir_deref_instr *deref = *deref_ptr;
+      switch (deref->deref_type) {
+      case nir_deref_type_array: {
+         unsigned elem_size =
+            glsl_count_attribute_slots(deref->type, false);
+         if (nir_src_is_const(deref->arr.index)) {
+            location += elem_size * nir_src_as_uint(deref->arr.index);
+         } else {
+            unsigned array_elems =
+               glsl_get_length(deref_ptr[-1]->type);
+            for (unsigned i = 0; i < array_elems; i++) {
+               gather_usage_helper(deref_ptr + 1,
+                                   location + elem_size * i,
+                                   mask, usage_mask);
+            }
+            return;
+         }
+         break;
+      }
+      case nir_deref_type_struct: {
+         const struct glsl_type *parent_type =
+            deref_ptr[-1]->type;
+         unsigned index = deref->strct.index;
+         for (unsigned i = 0; i < index; i++) {
+            const struct glsl_type *ft = glsl_get_struct_field(parent_type, i);
+            location += glsl_count_attribute_slots(ft, false);
+         }
+         break;
+      }
+      default:
+         unreachable("Unhandled deref type in gather_components_used_helper");
+      }
+   }
+
+   usage_mask[location] |= mask & 0xf;
+   if (mask & 0xf0)
+      usage_mask[location + 1] |= (mask >> 4) & 0xf;
 }
 
 static void gather_usage(const nir_deref_instr *deref,
-			 uint8_t mask,
-			 uint8_t *usage_mask)
+                         uint8_t mask,
+                         uint8_t *usage_mask)
 {
-	nir_deref_path path;
-	nir_deref_path_init(&path, (nir_deref_instr *)deref, NULL);
-
-	unsigned location_frac = path.path[0]->var->data.location_frac;
-	if (glsl_type_is_64bit(deref->type)) {
-		uint8_t new_mask = 0;
-		for (unsigned i = 0; i < 4; i++) {
-			if (mask & (1 << i))
-				new_mask |= 0x3 << (2 * i);
-		}
-		mask = new_mask << location_frac;
-	} else {
-		mask <<= location_frac;
-		mask &= 0xf;
-	}
-
-	gather_usage_helper((const nir_deref_instr **)&path.path[1],
-			    path.path[0]->var->data.driver_location,
-			    mask, usage_mask);
-
-	nir_deref_path_finish(&path);
+   nir_deref_path path;
+   nir_deref_path_init(&path, (nir_deref_instr *)deref, NULL);
+
+   unsigned location_frac = path.path[0]->var->data.location_frac;
+   if (glsl_type_is_64bit(deref->type)) {
+      uint8_t new_mask = 0;
+      for (unsigned i = 0; i < 4; i++) {
+         if (mask & (1 << i))
+            new_mask |= 0x3 << (2 * i);
+      }
+      mask = new_mask << location_frac;
+   } else {
+      mask <<= location_frac;
+      mask &= 0xf;
+   }
+
+   gather_usage_helper((const nir_deref_instr **)&path.path[1],
+                       path.path[0]->var->data.driver_location,
+                       mask, usage_mask);
+
+   nir_deref_path_finish(&path);
 }
 
 static void gather_intrinsic_load_deref_info(const nir_shader *nir,
                                              const nir_intrinsic_instr *instr,
-					     const nir_deref_instr *deref,
+                                             const nir_deref_instr *deref,
                                              bool need_texcoord,
                                              nir_variable *var,
                                              struct tgsi_shader_info *info)



More information about the mesa-commit mailing list