[Mesa-dev] [PATCH 10/12] nir/intrinsics: Add more atomic_counter ops

Ian Romanick idr at freedesktop.org
Tue Jul 19 20:13:12 UTC 2016


From: Ian Romanick <ian.d.romanick at intel.com>

v2: Delete some stray debug code notice by Iago.

Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
Reviewed-by: Iago Toral Quiroga <itoral at igalia.com>
---
 src/compiler/glsl/glsl_to_nir.cpp                  | 39 +++++++++++++++++++---
 src/compiler/glsl/nir_intrinsic_map.py             |  8 +++++
 .../glsl/tests/get_intrinsic_opcode_test.cpp       |  8 +++++
 src/compiler/nir/nir_intrinsics.h                  | 14 ++++++++
 src/compiler/nir/nir_lower_atomics.c               | 38 +++++++++++++++++++++
 5 files changed, 102 insertions(+), 5 deletions(-)

diff --git a/src/compiler/glsl/glsl_to_nir.cpp b/src/compiler/glsl/glsl_to_nir.cpp
index 3b8424e..f2b8b05 100644
--- a/src/compiler/glsl/glsl_to_nir.cpp
+++ b/src/compiler/glsl/glsl_to_nir.cpp
@@ -616,11 +616,40 @@ nir_visitor::visit(ir_call *ir)
       switch (op) {
       case nir_intrinsic_atomic_counter_read_var:
       case nir_intrinsic_atomic_counter_inc_var:
-      case nir_intrinsic_atomic_counter_dec_var: {
-         ir_dereference *param =
-            (ir_dereference *) ir->actual_parameters.get_head();
-         instr->variables[0] = evaluate_deref(&instr->instr, param);
-         nir_ssa_dest_init(&instr->instr, &instr->dest, 1, 32, NULL);
+      case nir_intrinsic_atomic_counter_dec_var:
+      case nir_intrinsic_atomic_counter_add_var:
+      case nir_intrinsic_atomic_counter_min_var:
+      case nir_intrinsic_atomic_counter_max_var:
+      case nir_intrinsic_atomic_counter_and_var:
+      case nir_intrinsic_atomic_counter_or_var:
+      case nir_intrinsic_atomic_counter_xor_var:
+      case nir_intrinsic_atomic_counter_exchange_var:
+      case nir_intrinsic_atomic_counter_comp_swap_var: {
+         /* Set the counter variable dereference. */
+         exec_node *param = ir->actual_parameters.get_head();
+         ir_dereference *counter = (ir_dereference *)param;
+
+         instr->variables[0] = evaluate_deref(&instr->instr, counter);
+         param = param->get_next();
+
+         /* Set the intrinsic destination. */
+         if (ir->return_deref) {
+            nir_ssa_dest_init(&instr->instr, &instr->dest, 1, 32, NULL);
+         }
+
+         /* Set the intrinsic parameters. */
+         if (!param->is_tail_sentinel()) {
+            instr->src[0] =
+               nir_src_for_ssa(evaluate_rvalue((ir_dereference *)param));
+            param = param->get_next();
+         }
+
+         if (!param->is_tail_sentinel()) {
+            instr->src[1] =
+               nir_src_for_ssa(evaluate_rvalue((ir_dereference *)param));
+            param = param->get_next();
+         }
+
          nir_builder_instr_insert(&b, &instr->instr);
          break;
       }
diff --git a/src/compiler/glsl/nir_intrinsic_map.py b/src/compiler/glsl/nir_intrinsic_map.py
index 3c2c76c..59b394e 100644
--- a/src/compiler/glsl/nir_intrinsic_map.py
+++ b/src/compiler/glsl/nir_intrinsic_map.py
@@ -27,6 +27,14 @@ from mako.template import Template
 intrinsics = [("__intrinsic_atomic_read", ("nir_intrinsic_atomic_counter_read_var", None)),
               ("__intrinsic_atomic_increment", ("nir_intrinsic_atomic_counter_inc_var", None)),
               ("__intrinsic_atomic_predecrement", ("nir_intrinsic_atomic_counter_dec_var", None)),
+              ("__intrinsic_atomic_add", ("nir_intrinsic_atomic_counter_add_var", None)),
+              ("__intrinsic_atomic_min", ("nir_intrinsic_atomic_counter_min_var", None)),
+              ("__intrinsic_atomic_max", ("nir_intrinsic_atomic_counter_max_var", None)),
+              ("__intrinsic_atomic_and", ("nir_intrinsic_atomic_counter_and_var", None)),
+              ("__intrinsic_atomic_or", ("nir_intrinsic_atomic_counter_or_var", None)),
+              ("__intrinsic_atomic_xor", ("nir_intrinsic_atomic_counter_xor_var", None)),
+              ("__intrinsic_atomic_exchange", ("nir_intrinsic_atomic_counter_exchange_var", None)),
+              ("__intrinsic_atomic_comp_swap", ("nir_intrinsic_atomic_counter_comp_swap_var", None)),
               ("__intrinsic_image_load", ("nir_intrinsic_image_load", None)),
               ("__intrinsic_image_store", ("nir_intrinsic_image_store", None)),
               ("__intrinsic_image_atomic_add", ("nir_intrinsic_image_atomic_add", None)),
diff --git a/src/compiler/glsl/tests/get_intrinsic_opcode_test.cpp b/src/compiler/glsl/tests/get_intrinsic_opcode_test.cpp
index aeecf32..d270a03 100644
--- a/src/compiler/glsl/tests/get_intrinsic_opcode_test.cpp
+++ b/src/compiler/glsl/tests/get_intrinsic_opcode_test.cpp
@@ -45,6 +45,14 @@ static const struct test_vector {
    test_vector("__intrinsic_atomic_read", nir_intrinsic_atomic_counter_read_var),
    test_vector("__intrinsic_atomic_increment", nir_intrinsic_atomic_counter_inc_var),
    test_vector("__intrinsic_atomic_predecrement", nir_intrinsic_atomic_counter_dec_var),
+   test_vector("__intrinsic_atomic_add", nir_intrinsic_atomic_counter_add),
+   test_vector("__intrinsic_atomic_min", nir_intrinsic_atomic_counter_min),
+   test_vector("__intrinsic_atomic_max", nir_intrinsic_atomic_counter_max),
+   test_vector("__intrinsic_atomic_and", nir_intrinsic_atomic_counter_and),
+   test_vector("__intrinsic_atomic_or", nir_intrinsic_atomic_counter_or),
+   test_vector("__intrinsic_atomic_xor", nir_intrinsic_atomic_counter_xor),
+   test_vector("__intrinsic_atomic_exchange", nir_intrinsic_atomic_counter_exchange),
+   test_vector("__intrinsic_atomic_comp_swap", nir_intrinsic_atomic_counter_comp_swap),
    test_vector("__intrinsic_image_load", nir_intrinsic_image_load),
    test_vector("__intrinsic_image_store", nir_intrinsic_image_store),
    test_vector("__intrinsic_image_atomic_add", nir_intrinsic_image_atomic_add),
diff --git a/src/compiler/nir/nir_intrinsics.h b/src/compiler/nir/nir_intrinsics.h
index 0f2c5ec..70a1f5c 100644
--- a/src/compiler/nir/nir_intrinsics.h
+++ b/src/compiler/nir/nir_intrinsics.h
@@ -139,10 +139,24 @@ INTRINSIC(set_vertex_count, 1, ARR(1), false, 0, 0, 0, xx, xx, xx, 0)
 #define ATOMIC(name, flags) \
    INTRINSIC(name##_var, 0, ARR(0), true, 1, 1, 0, xx, xx, xx, flags) \
    INTRINSIC(name, 1, ARR(1), true, 1, 0, 1, BASE, xx, xx, flags)
+#define ATOMIC2(name) \
+   INTRINSIC(name##_var, 1, ARR(1), true, 1, 1, 0, xx, xx, xx, 0) \
+   INTRINSIC(name, 2, ARR(1, 1), true, 1, 0, 1, BASE, xx, xx, 0)
+#define ATOMIC3(name) \
+   INTRINSIC(name##_var, 2, ARR(1, 1), true, 1, 1, 0, xx, xx, xx, 0) \
+   INTRINSIC(name, 3, ARR(1, 1, 1), true, 1, 0, 1, BASE, xx, xx, 0)
 
 ATOMIC(atomic_counter_inc, 0)
 ATOMIC(atomic_counter_dec, 0)
 ATOMIC(atomic_counter_read, NIR_INTRINSIC_CAN_ELIMINATE)
+ATOMIC2(atomic_counter_add)
+ATOMIC2(atomic_counter_min)
+ATOMIC2(atomic_counter_max)
+ATOMIC2(atomic_counter_and)
+ATOMIC2(atomic_counter_or)
+ATOMIC2(atomic_counter_xor)
+ATOMIC2(atomic_counter_exchange)
+ATOMIC3(atomic_counter_comp_swap)
 
 /*
  * Image load, store and atomic intrinsics.
diff --git a/src/compiler/nir/nir_lower_atomics.c b/src/compiler/nir/nir_lower_atomics.c
index 04e1feb..583e2a5 100644
--- a/src/compiler/nir/nir_lower_atomics.c
+++ b/src/compiler/nir/nir_lower_atomics.c
@@ -54,6 +54,38 @@ lower_instr(nir_intrinsic_instr *instr,
       op = nir_intrinsic_atomic_counter_dec;
       break;
 
+   case nir_intrinsic_atomic_counter_add_var:
+      op = nir_intrinsic_atomic_counter_add;
+      break;
+
+   case nir_intrinsic_atomic_counter_min_var:
+      op = nir_intrinsic_atomic_counter_min;
+      break;
+
+   case nir_intrinsic_atomic_counter_max_var:
+      op = nir_intrinsic_atomic_counter_max;
+      break;
+
+   case nir_intrinsic_atomic_counter_and_var:
+      op = nir_intrinsic_atomic_counter_and;
+      break;
+
+   case nir_intrinsic_atomic_counter_or_var:
+      op = nir_intrinsic_atomic_counter_or;
+      break;
+
+   case nir_intrinsic_atomic_counter_xor_var:
+      op = nir_intrinsic_atomic_counter_xor;
+      break;
+
+   case nir_intrinsic_atomic_counter_exchange_var:
+      op = nir_intrinsic_atomic_counter_exchange;
+      break;
+
+   case nir_intrinsic_atomic_counter_comp_swap_var:
+      op = nir_intrinsic_atomic_counter_comp_swap;
+      break;
+
    default:
       return;
    }
@@ -120,6 +152,12 @@ lower_instr(nir_intrinsic_instr *instr,
    new_instr->src[0].is_ssa = true;
    new_instr->src[0].ssa = offset_def;
 
+   /* Copy the other sources, if any, from the original instruction to the new
+    * instruction.
+    */
+   for (unsigned i = 0; i < nir_intrinsic_infos[instr->intrinsic].num_srcs; i++)
+      new_instr->src[i + 1] = instr->src[i];
+
    if (instr->dest.is_ssa) {
       nir_ssa_dest_init(&new_instr->instr, &new_instr->dest,
                         instr->dest.ssa.num_components, 32, NULL);
-- 
2.5.5



More information about the mesa-dev mailing list