zink: refactor vertex-order emitting
authorErik Faye-Lund <erik.faye-lund@collabora.com>
Tue, 2 Feb 2021 12:00:29 +0000 (13:00 +0100)
committerMarge Bot <eric+marge@anholt.net>
Tue, 2 Feb 2021 16:00:37 +0000 (16:00 +0000)
Using a ternary expression inside the argument list avoids some
repetition, showing that the rest of the call is the same. This increase
readability a tad.

Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz@gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/8832>

src/gallium/drivers/zink/nir_to_spirv/nir_to_spirv.c

index 56fb381..df58483 100644 (file)
@@ -3484,10 +3484,9 @@ nir_to_spirv(struct nir_shader *s, const struct zink_so_info *so_info,
    case MESA_SHADER_TESS_EVAL:
       spirv_builder_emit_exec_mode(&ctx.builder, entry_point,
                                    get_primitive_mode(s->info.tess.primitive_mode));
-      if (s->info.tess.ccw)
-         spirv_builder_emit_exec_mode(&ctx.builder, entry_point, SpvExecutionModeVertexOrderCcw);
-      else
-         spirv_builder_emit_exec_mode(&ctx.builder, entry_point, SpvExecutionModeVertexOrderCw);
+      spirv_builder_emit_exec_mode(&ctx.builder, entry_point,
+                                   s->info.tess.ccw ? SpvExecutionModeVertexOrderCcw
+                                                    : SpvExecutionModeVertexOrderCw);
       spirv_builder_emit_exec_mode(&ctx.builder, entry_point,
                                    get_spacing(s->info.tess.spacing));
       if (s->info.tess.point_mode)