Mesa (master): vc4: Rename UNPACK_8* to UNPACK_8*_F.

Eric Anholt anholt at kemper.freedesktop.org
Mon Dec 15 22:33:32 UTC 2014


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

Author: Eric Anholt <eric at anholt.net>
Date:   Mon Dec 15 10:45:58 2014 -0800

vc4: Rename UNPACK_8* to UNPACK_8*_F.

There is an equivalent unpack function without conversion to float if you
use an integer operation instead.

---

 src/gallium/drivers/vc4/vc4_program.c           |    2 +-
 src/gallium/drivers/vc4/vc4_qir.c               |    8 ++++----
 src/gallium/drivers/vc4/vc4_qir.h               |   12 ++++++------
 src/gallium/drivers/vc4/vc4_qpu_emit.c          |   10 +++++-----
 src/gallium/drivers/vc4/vc4_register_allocate.c |    8 ++++----
 5 files changed, 20 insertions(+), 20 deletions(-)

diff --git a/src/gallium/drivers/vc4/vc4_program.c b/src/gallium/drivers/vc4/vc4_program.c
index 1362170..0bec265 100644
--- a/src/gallium/drivers/vc4/vc4_program.c
+++ b/src/gallium/drivers/vc4/vc4_program.c
@@ -1024,7 +1024,7 @@ emit_vertex_input(struct vc4_compile *c, int attr)
                         struct qreg vpm = vpm_reads[0];
                         if (desc->channel[swiz].type == UTIL_FORMAT_TYPE_SIGNED)
                                 vpm = qir_XOR(c, vpm, qir_uniform_ui(c, 0x80808080));
-                        result = qir_UNPACK_8(c, vpm, swiz);
+                        result = qir_UNPACK_8_F(c, vpm, swiz);
                 } else {
                         if (!format_warned) {
                                 fprintf(stderr,
diff --git a/src/gallium/drivers/vc4/vc4_qir.c b/src/gallium/drivers/vc4/vc4_qir.c
index c43b9b6..e83c036 100644
--- a/src/gallium/drivers/vc4/vc4_qir.c
+++ b/src/gallium/drivers/vc4/vc4_qir.c
@@ -99,10 +99,10 @@ static const struct qir_op_info qir_op_info[] = {
         [QOP_R4_UNPACK_B] = { "r4_unpack_b", 1, 1 },
         [QOP_R4_UNPACK_C] = { "r4_unpack_c", 1, 1 },
         [QOP_R4_UNPACK_D] = { "r4_unpack_d", 1, 1 },
-        [QOP_UNPACK_8A] = { "unpack_8a", 1, 1 },
-        [QOP_UNPACK_8B] = { "unpack_8b", 1, 1 },
-        [QOP_UNPACK_8C] = { "unpack_8c", 1, 1 },
-        [QOP_UNPACK_8D] = { "unpack_8d", 1, 1 },
+        [QOP_UNPACK_8A_F] = { "unpack_8a_f", 1, 1 },
+        [QOP_UNPACK_8B_F] = { "unpack_8b_f", 1, 1 },
+        [QOP_UNPACK_8C_F] = { "unpack_8c_f", 1, 1 },
+        [QOP_UNPACK_8D_F] = { "unpack_8d_f", 1, 1 },
 };
 
 static const char *
diff --git a/src/gallium/drivers/vc4/vc4_qir.h b/src/gallium/drivers/vc4/vc4_qir.h
index 0b76a2f..c612b28 100644
--- a/src/gallium/drivers/vc4/vc4_qir.h
+++ b/src/gallium/drivers/vc4/vc4_qir.h
@@ -109,10 +109,10 @@ enum qop {
         QOP_FRAG_W,
         QOP_FRAG_REV_FLAG,
 
-        QOP_UNPACK_8A,
-        QOP_UNPACK_8B,
-        QOP_UNPACK_8C,
-        QOP_UNPACK_8D,
+        QOP_UNPACK_8A_F,
+        QOP_UNPACK_8B_F,
+        QOP_UNPACK_8C_F,
+        QOP_UNPACK_8D_F,
 
         /** Texture x coordinate parameter write */
         QOP_TEX_S,
@@ -489,10 +489,10 @@ qir_SEL_X_0_COND(struct vc4_compile *c, int i)
 }
 
 static inline struct qreg
-qir_UNPACK_8(struct vc4_compile *c, struct qreg src, int i)
+qir_UNPACK_8_F(struct vc4_compile *c, struct qreg src, int i)
 {
         struct qreg t = qir_get_temp(c);
-        qir_emit(c, qir_inst(QOP_UNPACK_8A + i, t, src, c->undef));
+        qir_emit(c, qir_inst(QOP_UNPACK_8A_F + i, t, src, c->undef));
         return t;
 }
 
diff --git a/src/gallium/drivers/vc4/vc4_qpu_emit.c b/src/gallium/drivers/vc4/vc4_qpu_emit.c
index c4359e7..8c58f18 100644
--- a/src/gallium/drivers/vc4/vc4_qpu_emit.c
+++ b/src/gallium/drivers/vc4/vc4_qpu_emit.c
@@ -468,10 +468,10 @@ vc4_generate_code(struct vc4_context *vc4, struct vc4_compile *c)
 
                         break;
 
-                case QOP_UNPACK_8A:
-                case QOP_UNPACK_8B:
-                case QOP_UNPACK_8C:
-                case QOP_UNPACK_8D: {
+                case QOP_UNPACK_8A_F:
+                case QOP_UNPACK_8B_F:
+                case QOP_UNPACK_8C_F:
+                case QOP_UNPACK_8D_F: {
                         assert(src[0].mux == QPU_MUX_A);
 
                         /* And, since we're setting the pack bits, if the
@@ -484,7 +484,7 @@ vc4_generate_code(struct vc4_context *vc4, struct vc4_compile *c)
                         queue(c, qpu_a_FMAX(dst, src[0], src[0]));
                         *last_inst(c) |= QPU_SET_FIELD(QPU_UNPACK_8A +
                                                        (qinst->op -
-                                                        QOP_UNPACK_8A),
+                                                        QOP_UNPACK_8A_F),
                                                        QPU_UNPACK);
 
                         if (orig_dst.mux == QPU_MUX_A) {
diff --git a/src/gallium/drivers/vc4/vc4_register_allocate.c b/src/gallium/drivers/vc4/vc4_register_allocate.c
index 85f29e5..f48ce18 100644
--- a/src/gallium/drivers/vc4/vc4_register_allocate.c
+++ b/src/gallium/drivers/vc4/vc4_register_allocate.c
@@ -254,10 +254,10 @@ vc4_register_allocate(struct vc4_context *vc4, struct vc4_compile *c)
                                           vc4->reg_class_a);
                         break;
 
-                case QOP_UNPACK_8A:
-                case QOP_UNPACK_8B:
-                case QOP_UNPACK_8C:
-                case QOP_UNPACK_8D:
+                case QOP_UNPACK_8A_F:
+                case QOP_UNPACK_8B_F:
+                case QOP_UNPACK_8C_F:
+                case QOP_UNPACK_8D_F:
                         /* The unpack flags require an A-file src register. */
                         ra_set_node_class(g, temp_to_node[inst->src[0].index],
                                           vc4->reg_class_a);




More information about the mesa-commit mailing list