Mesa (master): nir: Fix printing of ~0 .locations.

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Thu Jan 16 23:47:34 UTC 2020


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

Author: Eric Anholt <eric at anholt.net>
Date:   Wed Dec 18 16:46:41 2019 -0800

nir: Fix printing of ~0 .locations.

I kept wondering what "429" meant in variable declarations, when it was
just a truncated ~0 snprintf.

Reviewed-by: Rob Clark <robdclark at chromium.org>
Reviewed-by: Alyssa Rosenzweig <alyssa.rosenzweig at collabora.com>
Tested-by: Marge Bot <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3423>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/3423>

---

 src/compiler/nir/nir_print.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/compiler/nir/nir_print.c b/src/compiler/nir/nir_print.c
index 5d81d3424f7..aa9ee670a70 100644
--- a/src/compiler/nir/nir_print.c
+++ b/src/compiler/nir/nir_print.c
@@ -562,8 +562,12 @@ print_var_decl(nir_variable *var, print_state *state)
       }
 
       if (!loc) {
-         snprintf(buf, sizeof(buf), "%u", var->data.location);
-         loc = buf;
+         if (var->data.location == ~0) {
+            loc = "~0";
+         } else {
+            snprintf(buf, sizeof(buf), "%u", var->data.location);
+            loc = buf;
+         }
       }
 
       /* For shader I/O vars that have been split to components or packed,



More information about the mesa-commit mailing list