Mesa (main): lima/gpir: clean up override-init warnings

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Fri Feb 11 22:03:19 UTC 2022


Module: Mesa
Branch: main
Commit: 823be63216a407c7819eb1e62e8a6896bbd72bbc
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=823be63216a407c7819eb1e62e8a6896bbd72bbc

Author: Erico Nunes <nunes.erico at gmail.com>
Date:   Sun Jan 30 09:48:00 2022 +0100

lima/gpir: clean up override-init warnings

Define gpir_op_unsupported as 0 so that we don't have to do the
initialization to -1.

Signed-off-by: Erico Nunes <nunes.erico at gmail.com>
Reviewed-by: Vasily Khoruzhick <anarsoul at gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14894>

---

 src/gallium/drivers/lima/ir/gp/gpir.h | 1 +
 src/gallium/drivers/lima/ir/gp/nir.c  | 5 +----
 src/gallium/drivers/lima/ir/gp/node.c | 3 +++
 3 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/gallium/drivers/lima/ir/gp/gpir.h b/src/gallium/drivers/lima/ir/gp/gpir.h
index 63f74caa0ed..135fd36eeb8 100644
--- a/src/gallium/drivers/lima/ir/gp/gpir.h
+++ b/src/gallium/drivers/lima/ir/gp/gpir.h
@@ -32,6 +32,7 @@
 
 /* list of operations that a node can do. */
 typedef enum {
+   gpir_op_unsupported = 0,
    gpir_op_mov,
 
    /* mul ops */
diff --git a/src/gallium/drivers/lima/ir/gp/nir.c b/src/gallium/drivers/lima/ir/gp/nir.c
index 4b1479a68fc..b21d17c349e 100644
--- a/src/gallium/drivers/lima/ir/gp/nir.c
+++ b/src/gallium/drivers/lima/ir/gp/nir.c
@@ -147,9 +147,6 @@ static gpir_node *gpir_node_find(gpir_block *block, nir_src *src,
 }
 
 static int nir_to_gpir_opcodes[nir_num_opcodes] = {
-   /* not supported */
-   [0 ... nir_last_opcode] = -1,
-
    [nir_op_fmul] = gpir_op_mul,
    [nir_op_fadd] = gpir_op_add,
    [nir_op_fneg] = gpir_op_neg,
@@ -188,7 +185,7 @@ static bool gpir_emit_alu(gpir_block *block, nir_instr *ni)
 
    int op = nir_to_gpir_opcodes[instr->op];
 
-   if (op < 0) {
+   if (op == gpir_op_unsupported) {
       gpir_error("unsupported nir_op: %s\n", nir_op_infos[instr->op].name);
       return false;
    }
diff --git a/src/gallium/drivers/lima/ir/gp/node.c b/src/gallium/drivers/lima/ir/gp/node.c
index ef534e7e286..9372d72a8bf 100644
--- a/src/gallium/drivers/lima/ir/gp/node.c
+++ b/src/gallium/drivers/lima/ir/gp/node.c
@@ -28,6 +28,9 @@
 #include "gpir.h"
 
 const gpir_op_info gpir_op_infos[] = {
+   [gpir_op_unsupported] = {
+      .name = "unsupported",
+   },
    [gpir_op_mov] = {
       .name = "mov",
       .slots = (int []) {



More information about the mesa-commit mailing list