[Mesa-dev] [PATCH] nir: delete magic number

Elie Tournier tournier.elie at gmail.com
Wed Feb 8 16:56:40 UTC 2017


Explaining what is the "26" for.

PS: I don't have git rights. Can you push this patch for me?

Signed-off-by: Elie Tournier <tournier.elie at gmail.com>
---
 src/compiler/nir/nir_opt_loop_unroll.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/compiler/nir/nir_opt_loop_unroll.c b/src/compiler/nir/nir_opt_loop_unroll.c
index 37cbced43d..2bb1a57791 100644
--- a/src/compiler/nir/nir_opt_loop_unroll.c
+++ b/src/compiler/nir/nir_opt_loop_unroll.c
@@ -26,6 +26,8 @@
 #include "nir_control_flow.h"
 #include "nir_loop_analyze.h"
 
+#define GLSL_IR_UNROLL_LIMIT 26
+
 /* Prepare this loop for unrolling by first converting to lcssa and then
  * converting the phis from the loops first block and the block that follows
  * the loop into regs.  Partially converting out of SSA allows us to unroll
@@ -460,7 +462,7 @@ is_loop_small_enough_to_unroll(nir_shader *shader, nir_loop_info *li)
       return true;
 
    bool loop_not_too_large =
-      li->num_instructions * li->trip_count <= max_iter * 26;
+      li->num_instructions * li->trip_count <= max_iter * GLSL_IR_UNROLL_LIMIT;
 
    return loop_not_too_large;
 }
-- 
2.11.0



More information about the mesa-dev mailing list