From 193e2c9625b5c9830d6f1f5a9fcf0fe77ebee378 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marek=20Ol=C5=A1=C3=A1k?= <marek.olsak@amd.com>
Date: Thu, 7 Nov 2019 17:54:13 -0500
Subject: [PATCH] nir/print: only print image.format for image variables

Reviewed-by: Connor Abbott <cwabbott0@gmail.com>
---
 src/compiler/nir/nir_print.c | 90 ++++++++++++++++++------------------
 1 file changed, 46 insertions(+), 44 deletions(-)

diff --git a/src/compiler/nir/nir_print.c b/src/compiler/nir/nir_print.c
index 8dbf701f8ef..ab82f4fbb50 100644
--- a/src/compiler/nir/nir_print.c
+++ b/src/compiler/nir/nir_print.c
@@ -467,52 +467,54 @@ print_var_decl(nir_variable *var, print_state *state)
    const char *const reorder = (access & ACCESS_CAN_REORDER) ? "reorderable " : "";
    fprintf(fp, "%s%s%s%s%s%s", coher, volat, restr, ronly, wonly, reorder);
 
+   if (glsl_get_base_type(glsl_without_array(var->type)) == GLSL_TYPE_IMAGE) {
 #define FORMAT_CASE(x) case x: fprintf(fp, #x " "); break
-   switch (var->data.image.format) {
-   FORMAT_CASE(GL_RGBA32F);
-   FORMAT_CASE(GL_RGBA32UI);
-   FORMAT_CASE(GL_RGBA32I);
-   FORMAT_CASE(GL_R32F);
-   FORMAT_CASE(GL_R32UI);
-   FORMAT_CASE(GL_R32I);
-   FORMAT_CASE(GL_RG32F);
-   FORMAT_CASE(GL_RG32UI);
-   FORMAT_CASE(GL_RG32I);
-   FORMAT_CASE(GL_R8);
-   FORMAT_CASE(GL_RG8);
-   FORMAT_CASE(GL_RGBA8);
-   FORMAT_CASE(GL_R8_SNORM);
-   FORMAT_CASE(GL_RG8_SNORM);
-   FORMAT_CASE(GL_RGBA8_SNORM);
-   FORMAT_CASE(GL_R16);
-   FORMAT_CASE(GL_RG16);
-   FORMAT_CASE(GL_RGBA16);
-   FORMAT_CASE(GL_R16_SNORM);
-   FORMAT_CASE(GL_RG16_SNORM);
-   FORMAT_CASE(GL_RGBA16_SNORM);
-   FORMAT_CASE(GL_R16F);
-   FORMAT_CASE(GL_RG16F);
-   FORMAT_CASE(GL_RGBA16F);
-   FORMAT_CASE(GL_R8UI);
-   FORMAT_CASE(GL_R8I);
-   FORMAT_CASE(GL_RG8UI);
-   FORMAT_CASE(GL_RG8I);
-   FORMAT_CASE(GL_RGBA8UI);
-   FORMAT_CASE(GL_RGBA8I);
-   FORMAT_CASE(GL_R16UI);
-   FORMAT_CASE(GL_R16I);
-   FORMAT_CASE(GL_RG16UI);
-   FORMAT_CASE(GL_RG16I);
-   FORMAT_CASE(GL_RGBA16UI);
-   FORMAT_CASE(GL_RGBA16I);
-   FORMAT_CASE(GL_R11F_G11F_B10F);
-   FORMAT_CASE(GL_RGB9_E5);
-   FORMAT_CASE(GL_RGB10_A2);
-   FORMAT_CASE(GL_RGB10_A2UI);
-   default: /* Including the normal GL_NONE */
-      break;
-   }
+      switch (var->data.image.format) {
+      FORMAT_CASE(GL_RGBA32F);
+      FORMAT_CASE(GL_RGBA32UI);
+      FORMAT_CASE(GL_RGBA32I);
+      FORMAT_CASE(GL_R32F);
+      FORMAT_CASE(GL_R32UI);
+      FORMAT_CASE(GL_R32I);
+      FORMAT_CASE(GL_RG32F);
+      FORMAT_CASE(GL_RG32UI);
+      FORMAT_CASE(GL_RG32I);
+      FORMAT_CASE(GL_R8);
+      FORMAT_CASE(GL_RG8);
+      FORMAT_CASE(GL_RGBA8);
+      FORMAT_CASE(GL_R8_SNORM);
+      FORMAT_CASE(GL_RG8_SNORM);
+      FORMAT_CASE(GL_RGBA8_SNORM);
+      FORMAT_CASE(GL_R16);
+      FORMAT_CASE(GL_RG16);
+      FORMAT_CASE(GL_RGBA16);
+      FORMAT_CASE(GL_R16_SNORM);
+      FORMAT_CASE(GL_RG16_SNORM);
+      FORMAT_CASE(GL_RGBA16_SNORM);
+      FORMAT_CASE(GL_R16F);
+      FORMAT_CASE(GL_RG16F);
+      FORMAT_CASE(GL_RGBA16F);
+      FORMAT_CASE(GL_R8UI);
+      FORMAT_CASE(GL_R8I);
+      FORMAT_CASE(GL_RG8UI);
+      FORMAT_CASE(GL_RG8I);
+      FORMAT_CASE(GL_RGBA8UI);
+      FORMAT_CASE(GL_RGBA8I);
+      FORMAT_CASE(GL_R16UI);
+      FORMAT_CASE(GL_R16I);
+      FORMAT_CASE(GL_RG16UI);
+      FORMAT_CASE(GL_RG16I);
+      FORMAT_CASE(GL_RGBA16UI);
+      FORMAT_CASE(GL_RGBA16I);
+      FORMAT_CASE(GL_R11F_G11F_B10F);
+      FORMAT_CASE(GL_RGB9_E5);
+      FORMAT_CASE(GL_RGB10_A2);
+      FORMAT_CASE(GL_RGB10_A2UI);
+      default: /* Including the normal GL_NONE */
+         break;
+      }
 #undef FORMAT_CASE
+   }
 
    fprintf(fp, "%s %s", glsl_get_type_name(var->type),
            get_var_name(var, state));