nir: make is_comparison() a non-static helper function
authorSamuel Pitoiset <samuel.pitoiset@gmail.com>
Wed, 30 May 2018 12:21:41 +0000 (14:21 +0200)
committerSamuel Pitoiset <samuel.pitoiset@gmail.com>
Mon, 4 Jun 2018 10:41:08 +0000 (12:41 +0200)
Rename and change the prototype for consistency regarding
nir_tex_instr_is_query(). This function will be used in the
following patch.

Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
src/compiler/nir/nir.h
src/compiler/nir/nir_opt_move_comparisons.c

index f6086bd6c058a40e07ec00839a5491493dbc6459..5a1f79515ad63af8fc6154804ed02bfe8fac2a53 100644 (file)
@@ -1374,6 +1374,30 @@ nir_tex_instr_is_query(const nir_tex_instr *instr)
    }
 }
 
+static inline bool
+nir_alu_instr_is_comparison(const nir_alu_instr *instr)
+{
+   switch (instr->op) {
+   case nir_op_flt:
+   case nir_op_fge:
+   case nir_op_feq:
+   case nir_op_fne:
+   case nir_op_ilt:
+   case nir_op_ult:
+   case nir_op_ige:
+   case nir_op_uge:
+   case nir_op_ieq:
+   case nir_op_ine:
+   case nir_op_i2b:
+   case nir_op_f2b:
+   case nir_op_inot:
+   case nir_op_fnot:
+      return true;
+   default:
+      return false;
+   }
+}
+
 static inline nir_alu_type
 nir_tex_instr_src_type(const nir_tex_instr *instr, unsigned src)
 {
index 617c2ca99897f5f9eb569adac0ce29f090d2a832..5da57dc9213b516e2ae2542191b7df36e88a25de 100644 (file)
  * lower register pressure.
  */
 
-static bool
-is_comparison(nir_op op)
-{
-   switch (op) {
-   case nir_op_flt:
-   case nir_op_fge:
-   case nir_op_feq:
-   case nir_op_fne:
-   case nir_op_ilt:
-   case nir_op_ult:
-   case nir_op_ige:
-   case nir_op_uge:
-   case nir_op_ieq:
-   case nir_op_ine:
-   case nir_op_i2b:
-   case nir_op_f2b:
-   case nir_op_inot:
-   case nir_op_fnot:
-      return true;
-   default:
-      return false;
-   }
-}
-
 static bool
 move_comparison_source(nir_src *src, nir_block *block, nir_instr *before)
 {
@@ -84,7 +60,7 @@ move_comparison_source(nir_src *src, nir_block *block, nir_instr *before)
 
    if (src_instr->block == block &&
        src_instr->type == nir_instr_type_alu &&
-       is_comparison(nir_instr_as_alu(src_instr)->op)) {
+       nir_alu_instr_is_comparison(nir_instr_as_alu(src_instr))) {
 
       exec_node_remove(&src_instr->node);