glsl/loops: Get rid of lower_bounded_loops and ir_loop::normative_bound.
authorPaul Berry <stereotype441@gmail.com>
Fri, 29 Nov 2013 08:52:11 +0000 (00:52 -0800)
committerPaul Berry <stereotype441@gmail.com>
Mon, 9 Dec 2013 18:55:09 +0000 (10:55 -0800)
Now that loop_controls no longer creates normatively bound loops,
there is no need for ir_loop::normative_bound or the
lower_bounded_loops pass.

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
45 files changed:
src/glsl/Makefile.sources
src/glsl/ir.cpp
src/glsl/ir.h
src/glsl/ir_clone.cpp
src/glsl/ir_optimization.h
src/glsl/ir_print_visitor.cpp
src/glsl/ir_reader.cpp
src/glsl/loop_controls.cpp
src/glsl/loop_unroll.cpp
src/glsl/lower_bounded_loops.cpp [deleted file]
src/glsl/tests/lower_jumps/lower_breaks_1.opt_test
src/glsl/tests/lower_jumps/lower_breaks_1.opt_test.expected
src/glsl/tests/lower_jumps/lower_breaks_2.opt_test
src/glsl/tests/lower_jumps/lower_breaks_2.opt_test.expected
src/glsl/tests/lower_jumps/lower_breaks_3.opt_test
src/glsl/tests/lower_jumps/lower_breaks_3.opt_test.expected
src/glsl/tests/lower_jumps/lower_breaks_4.opt_test
src/glsl/tests/lower_jumps/lower_breaks_4.opt_test.expected
src/glsl/tests/lower_jumps/lower_breaks_5.opt_test
src/glsl/tests/lower_jumps/lower_breaks_5.opt_test.expected
src/glsl/tests/lower_jumps/lower_breaks_6.opt_test
src/glsl/tests/lower_jumps/lower_breaks_6.opt_test.expected
src/glsl/tests/lower_jumps/lower_guarded_conditional_break.opt_test
src/glsl/tests/lower_jumps/lower_guarded_conditional_break.opt_test.expected
src/glsl/tests/lower_jumps/lower_pulled_out_jump.opt_test
src/glsl/tests/lower_jumps/lower_pulled_out_jump.opt_test.expected
src/glsl/tests/lower_jumps/remove_continue_at_end_of_loop.opt_test
src/glsl/tests/lower_jumps/remove_continue_at_end_of_loop.opt_test.expected
src/glsl/tests/lower_jumps/return_non_void_at_end_of_loop_lower_nothing.opt_test
src/glsl/tests/lower_jumps/return_non_void_at_end_of_loop_lower_nothing.opt_test.expected
src/glsl/tests/lower_jumps/return_non_void_at_end_of_loop_lower_return.opt_test
src/glsl/tests/lower_jumps/return_non_void_at_end_of_loop_lower_return.opt_test.expected
src/glsl/tests/lower_jumps/return_non_void_at_end_of_loop_lower_return_and_break.opt_test
src/glsl/tests/lower_jumps/return_non_void_at_end_of_loop_lower_return_and_break.opt_test.expected
src/glsl/tests/lower_jumps/return_void_at_end_of_loop_lower_nothing.opt_test
src/glsl/tests/lower_jumps/return_void_at_end_of_loop_lower_nothing.opt_test.expected
src/glsl/tests/lower_jumps/return_void_at_end_of_loop_lower_return.opt_test
src/glsl/tests/lower_jumps/return_void_at_end_of_loop_lower_return.opt_test.expected
src/glsl/tests/lower_jumps/return_void_at_end_of_loop_lower_return_and_break.opt_test
src/glsl/tests/lower_jumps/return_void_at_end_of_loop_lower_return_and_break.opt_test.expected
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
src/mesa/drivers/dri/i965/brw_shader.cpp
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
src/mesa/program/ir_to_mesa.cpp
src/mesa/state_tracker/st_glsl_to_tgsi.cpp

index 724c7b403caa14f462f4c247b612ac6467493dc4..2e81deddabb9c79ee4a6a0ff78f38c14cdacab38 100644 (file)
@@ -59,7 +59,6 @@ LIBGLSL_FILES = \
        $(GLSL_SRCDIR)/loop_analysis.cpp \
        $(GLSL_SRCDIR)/loop_controls.cpp \
        $(GLSL_SRCDIR)/loop_unroll.cpp \
-       $(GLSL_SRCDIR)/lower_bounded_loops.cpp \
        $(GLSL_SRCDIR)/lower_clip_distance.cpp \
        $(GLSL_SRCDIR)/lower_discard.cpp \
        $(GLSL_SRCDIR)/lower_discard_flow.cpp \
index 29fe64ad496ed98ed2daa77b3beb6cf9f3594246..623bc1bb2e97398448ed57f47cb1f2f7ce43b541 100644 (file)
@@ -1277,7 +1277,6 @@ ir_constant::is_basis() const
 ir_loop::ir_loop()
 {
    this->ir_type = ir_type_loop;
-   this->normative_bound = -1;
 }
 
 
index a34a9c3193b70056af29da912983c4a6a9ca1611..7a939a236b8331e2a8058d6e9db23cd5f25199f6 100644 (file)
@@ -1024,13 +1024,6 @@ public:
 
    /** List of ir_instruction that make up the body of the loop. */
    exec_list body_instructions;
-
-   /**
-    * Normative bound for the loop.  If this value is >= 0, the back-end
-    * should generate instructions to ensure that the loop executes no more
-    * than this many times.
-    */
-   int normative_bound;
 };
 
 
index 383627ae3e22c244cede59f197bacc3b9972d8b9..df6e971f41a0d5f251fd710346e8b0a8b166695e 100644 (file)
@@ -158,8 +158,6 @@ ir_loop::clone(void *mem_ctx, struct hash_table *ht) const
 {
    ir_loop *new_loop = new(mem_ctx) ir_loop();
 
-   new_loop->normative_bound = this->normative_bound;
-
    foreach_iter(exec_list_iterator, iter, this->body_instructions) {
       ir_instruction *ir = (ir_instruction *)iter.get();
       new_loop->body_instructions.push_tail(ir->clone(mem_ctx, ht));
index 87c2c820b3c9357a794ee91d2f5f1751b863dbb1..3ca9f574453c32e705aa5889c3a8f5959edc878e 100644 (file)
@@ -101,7 +101,6 @@ bool do_swizzle_swizzle(exec_list *instructions);
 bool do_tree_grafting(exec_list *instructions);
 bool do_vec_index_to_cond_assign(exec_list *instructions);
 bool do_vec_index_to_swizzle(exec_list *instructions);
-bool lower_bounded_loops(exec_list *instructions);
 bool lower_discard(exec_list *instructions);
 void lower_discard_flow(exec_list *instructions);
 bool lower_instructions(exec_list *instructions, unsigned what_to_lower);
index 857a27ad21f3d71b551ace6e9c4cfcf5f92158cd..2beefedf16ee1b83f27a0c93cf2a02ad56351db1 100644 (file)
@@ -523,10 +523,7 @@ ir_print_visitor::visit(ir_if *ir)
 void
 ir_print_visitor::visit(ir_loop *ir)
 {
-   printf("(loop (");
-   if (ir->normative_bound >= 0)
-      printf("%d", ir->normative_bound);
-   printf(") (\n");
+   printf("(loop (\n");
    indentation++;
 
    foreach_iter(exec_list_iterator, iter, ir->body_instructions) {
index 72ce684b69cb2e06b3c59c3a937c18f9fea473d3..4afe28101390bc831b6825f3e7afc39df9a5bb44 100644 (file)
@@ -488,34 +488,16 @@ ir_reader::read_if(s_expression *expr, ir_loop *loop_ctx)
 ir_loop *
 ir_reader::read_loop(s_expression *expr)
 {
-   s_expression *s_bound_expr, *s_body, *s_bound;
+   s_expression *s_body;
 
-   s_pattern loop_pat[] = { "loop", s_bound_expr, s_body };
-   s_pattern no_bound_pat[] = { };
-   s_pattern bound_pat[] = { s_bound };
+   s_pattern loop_pat[] = { "loop", s_body };
    if (!MATCH(expr, loop_pat)) {
-      ir_read_error(expr, "expected (loop <bound> <body>)");
+      ir_read_error(expr, "expected (loop <body>)");
       return NULL;
    }
 
    ir_loop *loop = new(mem_ctx) ir_loop;
 
-   if (MATCH(s_bound_expr, no_bound_pat)) {
-      loop->normative_bound = -1;
-   } else if (MATCH(s_bound_expr, bound_pat)) {
-      s_int *value = SX_AS_INT(s_bound);
-      if (value == NULL) {
-         ir_read_error(s_bound_expr, "malformed loop bound");
-         delete loop;
-         return NULL;
-      }
-      loop->normative_bound = value->value();
-   } else {
-      ir_read_error(s_bound_expr, "malformed loop bound");
-      delete loop;
-      return NULL;
-   }
-
    read_instructions(&loop->body_instructions, s_body, loop);
    if (state->error) {
       delete loop;
index 385c2031c421e4205d178e7bb2acbc62926b714d..3db06ad18c1f0fe767ffac425d890d8b21ba14a1 100644 (file)
@@ -193,13 +193,6 @@ loop_control_visitor::visit_leave(ir_loop *ir)
          this->progress = true;
          return visit_continue;
       }
-
-      /* If the limiting terminator has a lower iteration count than the
-       * normative loop bound (if any), then the loop doesn't need a normative
-       * bound anymore.
-       */
-      if (ir->normative_bound >= 0 && iterations < ir->normative_bound)
-         ir->normative_bound = -1;
    }
 
    /* Remove the conditional break statements associated with all terminators
@@ -215,7 +208,7 @@ loop_control_visitor::visit_leave(ir_loop *ir)
       if (t->iterations < 0)
          continue;
 
-      if (ir->normative_bound >= 0 || t != ls->limiting_terminator) {
+      if (t != ls->limiting_terminator) {
          t->ir->remove();
 
          assert(ls->num_loop_jumps > 0);
index 4645dcbab199f18d52ca06504d883dbffea5f713..6eced17365b859f1276a2227be90cfc9a8c885b6 100644 (file)
@@ -228,9 +228,6 @@ loop_unroll_visitor::visit_leave(ir_loop *ir)
    loop_variable_state *const ls = this->state->get(ir);
    int iterations;
 
-   /* Note: normatively-bounded loops aren't created anymore. */
-   assert(ir->normative_bound < 0);
-
    /* If we've entered a loop that hasn't been analyzed, something really,
     * really bad has happened.
     */
diff --git a/src/glsl/lower_bounded_loops.cpp b/src/glsl/lower_bounded_loops.cpp
deleted file mode 100644 (file)
index 0cd907d..0000000
+++ /dev/null
@@ -1,117 +0,0 @@
-/*
- * Copyright © 2013 Intel Corporation
- *
- * Permission is hereby granted, free of charge, to any person obtaining a
- * copy of this software and associated documentation files (the "Software"),
- * to deal in the Software without restriction, including without limitation
- * the rights to use, copy, modify, merge, publish, distribute, sublicense,
- * and/or sell copies of the Software, and to permit persons to whom the
- * Software is furnished to do so, subject to the following conditions:
- *
- * The above copyright notice and this permission notice (including the next
- * paragraph) shall be included in all copies or substantial portions of the
- * Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
- * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
- * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
- * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
- * DEALINGS IN THE SOFTWARE.
- */
-
-/** \file
- *
- * This pass converts bounded loops (those whose ir_loop contains non-null
- * values for \c from, \c to, \c increment, and \c counter) into unbounded
- * loops.
- *
- * For instance:
- *
- * (loop (declare () uint i) (constant uint 0) (constant uint 4)
- *       (constant uint 1)
- *   ...loop body...)
- *
- * Is transformed into:
- *
- * (declare () uint i)
- * (assign (x) (var_ref i) (constant uint 0))
- * (loop
- *   (if (expression bool >= (var_ref i) (constant uint 4))
- *       (break)
- *     ())
- *   ...loop body...
- *   (assign (x) (var_ref i)
- *               (expression uint + (var_ref i) (constant uint 1))))
- */
-
-#include "ir_hierarchical_visitor.h"
-#include "ir.h"
-#include "ir_builder.h"
-
-using namespace ir_builder;
-
-namespace {
-
-class lower_bounded_loops_visitor : public ir_hierarchical_visitor {
-public:
-   lower_bounded_loops_visitor()
-      : progress(false)
-   {
-   }
-
-   virtual ir_visitor_status visit_leave(ir_loop *ir);
-
-   bool progress;
-};
-
-} /* anonymous namespace */
-
-
-ir_visitor_status
-lower_bounded_loops_visitor::visit_leave(ir_loop *ir)
-{
-   if (ir->normative_bound < 0)
-      return visit_continue;
-
-   exec_list new_instructions;
-   ir_factory f(&new_instructions, ralloc_parent(ir));
-
-   /* Before the loop, declare the counter and initialize it to zero. */
-   ir_variable *counter = f.make_temp(glsl_type::uint_type, "counter");
-   f.emit(assign(counter, f.constant(0u)));
-   ir->insert_before(&new_instructions);
-
-   /* At the top of the loop, compare the counter to normative_bound, and
-    * break if the comparison succeeds.
-    */
-   ir_loop_jump *brk = new(f.mem_ctx) ir_loop_jump(ir_loop_jump::jump_break);
-   ir_if *if_inst = if_tree(gequal(counter,
-                                   f.constant((unsigned) ir->normative_bound)),
-                            brk);
-   ir->body_instructions.push_head(if_inst);
-
-   /* At the bottom of the loop, increment the counter. */
-   ir->body_instructions.push_tail(assign(counter,
-                                          add(counter, f.constant(1u))));
-
-   /* Since we've explicitly added instructions to terminate the loop, we no
-    * longer need it to have a normative bound.
-    */
-   ir->normative_bound = -1;
-
-   this->progress = true;
-   return visit_continue;
-}
-
-
-bool
-lower_bounded_loops(exec_list *instructions)
-{
-   lower_bounded_loops_visitor v;
-
-   visit_list_elements(&v, instructions);
-
-   return v.progress;
-}
index c38ed9e70c6be008dd9535770cea5122d9aae307..b412ba8f29f5f82c5bde9e0d820277ff2bb08a4f 100755 (executable)
@@ -8,6 +8,6 @@
 ((declare (out) float a)
  (function main
   (signature void (parameters)
-   ((loop ()
+   ((loop
      ((assign (x) (var_ref a) (constant float (1.000000))) break))))))
 EOF
index b278b1c79c1c0b85c769aca91bd78b41b706c5f5..56ef3e4246537d44cb74308bd299ea6901da2d67 100644 (file)
@@ -1,5 +1,5 @@
 ((declare (out) float a)
  (function main
   (signature void (parameters)
-   ((loop ()
+   ((loop
      ((assign (x) (var_ref a) (constant float (1.000000))) break))))))
index a4a94e05be57c0cee4ba91b36b13e8f1274337c7..f5de80339cdf4583252eb1fb9f3556936024a939 100755 (executable)
@@ -8,7 +8,7 @@
 ((declare (in) float b) (declare (out) float a)
  (function main
   (signature void (parameters)
-   ((loop ()
+   ((loop
      ((assign (x) (var_ref a) (constant float (1.000000)))
       (if (expression bool > (var_ref b) (constant float (0.000000))) (break)
        ())))))))
index 90156be830481d6838a37ae964f296fb08bb0950..dc231f9753b689a0185fe7380bbd993758d62dbc 100644 (file)
@@ -1,7 +1,7 @@
 ((declare (in) float b) (declare (out) float a)
  (function main
   (signature void (parameters)
-   ((loop ()
+   ((loop
      ((assign (x) (var_ref a) (constant float (1.000000)))
       (if (expression bool > (var_ref b) (constant float (0.0))) (break)
        ())))))))
index 4475eea0b359fcf18e6687f01973e4c0320d82ad..60368bc1dc9194aaed7701861372d62c0e9092e7 100755 (executable)
@@ -9,7 +9,7 @@
 ((declare (in) float b) (declare (out) float a) (declare (out) float c)
  (function main
   (signature void (parameters)
-   ((loop ()
+   ((loop
      ((assign (x) (var_ref a) (constant float (1.000000)))
       (if (expression bool > (var_ref b) (constant float (0.000000)))
        ((assign (x) (var_ref c) (constant float (1.000000))) break)
index e31a71257de9bd5b37e5eec1e6d79498c647b1f0..8131b66ff4b03fa374f087283eca8d2ac22a66db 100644 (file)
@@ -1,7 +1,7 @@
 ((declare (in) float b) (declare (out) float a) (declare (out) float c)
  (function main
   (signature void (parameters)
-   ((loop ()
+   ((loop
      ((assign (x) (var_ref a) (constant float (1.000000)))
       (if (expression bool > (var_ref b) (constant float (0.0)))
        ((assign (x) (var_ref c) (constant float (1.000000))) break)
index cb06cfd6ddd2b975c2134e933956aa9c28425cc0..cde319793baadd03bfaf6eb62a51d4464d566dda 100755 (executable)
@@ -8,7 +8,7 @@
 ((declare (in) float b) (declare (out) float a)
  (function main
   (signature void (parameters)
-   ((loop ()
+   ((loop
      ((assign (x) (var_ref a) (constant float (1.000000)))
       (if (expression bool > (var_ref b) (constant float (0.000000))) ()
        (break))))))))
index 3c216ed6a573d8ad4f134b15c6630ddafe5fdadf..94dcb37be04eaa6e2c835fc4f5ad73dcf02635c8 100644 (file)
@@ -1,7 +1,7 @@
 ((declare (in) float b) (declare (out) float a)
  (function main
   (signature void (parameters)
-   ((loop ()
+   ((loop
      ((assign (x) (var_ref a) (constant float (1.000000)))
       (if (expression bool > (var_ref b) (constant float (0.0))) ()
        (break))))))))
index 0a7a11250f2fd76009ae124ffdc156b773bb9791..157b5892ffe8e132f916d2d6292db538ae7e30c7 100755 (executable)
@@ -9,7 +9,7 @@
 ((declare (in) float b) (declare (out) float a) (declare (out) float c)
  (function main
   (signature void (parameters)
-   ((loop ()
+   ((loop
      ((assign (x) (var_ref a) (constant float (1.000000)))
       (if (expression bool > (var_ref b) (constant float (0.000000))) ()
        ((assign (x) (var_ref c) (constant float (1.000000))) break))))))))
index 45e718ddcd618b008e9203c8cd22e9e0ed1702ee..5b46ccb677331d26610c9a7207c51370bbf9dec3 100644 (file)
@@ -1,7 +1,7 @@
 ((declare (in) float b) (declare (out) float a) (declare (out) float c)
  (function main
   (signature void (parameters)
-   ((loop ()
+   ((loop
      ((assign (x) (var_ref a) (constant float (1.000000)))
       (if (expression bool > (var_ref b) (constant float (0.0))) ()
        ((assign (x) (var_ref c) (constant float (1.000000))) break))))))))
index 4786063056799eaec8a6341c600993b2f0f03fc6..4767df13ed42f0979d4d444acae6199a9554e847 100755 (executable)
@@ -12,7 +12,7 @@
  (declare (in) float cb)
  (function main
   (signature void (parameters)
-   ((loop ()
+   ((loop
      ((if (expression bool > (var_ref a) (constant float (0.000000)))
        ((if (expression bool > (var_ref ba) (constant float (0.000000)))
          ((if (expression bool > (var_ref bb) (constant float (0.000000)))
index 338d751e84d52156f69ec0daaec10380fdf07dbd..967ce642a4b2d8a291fb21e64d10d3980ec139b9 100644 (file)
@@ -5,7 +5,7 @@
   (signature void (parameters)
    ((declare (temporary) bool break_flag)
     (assign (x) (var_ref break_flag) (constant bool (0)))
-    (loop ()
+    (loop
      ((declare (temporary) bool execute_flag)
       (assign (x) (var_ref execute_flag) (constant bool (1)))
       (if (expression bool > (var_ref a) (constant float (0.0)))
index 0f84c5d507c72d04e377189138e45d3ccd5d15be..164914a4294dcf5ec38a45701ed48569a31ad99c 100755 (executable)
@@ -10,7 +10,7 @@
 ((declare (in) float aa) (declare (in) float ab) (declare (in) float b)
  (function main
   (signature void (parameters)
-   ((loop ()
+   ((loop
      ((if (expression bool > (var_ref aa) (constant float (0.000000)))
        ((if (expression bool > (var_ref ab) (constant float (0.000000)))
          (continue)
index 3ce323aead8662d64db20375165a68a98cd725d8..841073ed909de7d9b1fc3319f59c8c889ce22e40 100644 (file)
@@ -3,7 +3,7 @@
   (signature void (parameters)
    ((declare (temporary) bool break_flag)
     (assign (x) (var_ref break_flag) (constant bool (0)))
-    (loop ()
+    (loop
      ((declare (temporary) bool execute_flag)
       (assign (x) (var_ref execute_flag) (constant bool (1)))
       (if (expression bool > (var_ref aa) (constant float (0.0)))
index cebdad78d36001132318175c9168d12cedd50f65..1a5c09690bf0e8e13dd9a9725e992f8b248616fb 100755 (executable)
@@ -19,7 +19,7 @@
        ((return))
        ()))
      ())
-    (loop ()
+    (loop
      ((if (expression bool > (var_ref b) (constant float (0.000000)))
        ((if (expression bool > (var_ref c) (constant float (0.000000))) (break)
          (continue)))
index e4339b14268bbf68ae36305ae0463a1b39f02e3e..cf2ef3f080a67aeb83acf147a824f56087feda50 100644 (file)
@@ -14,7 +14,7 @@
        ()))
      ())
     (if (var_ref execute_flag)
-     ((loop ()
+     ((loop
        ((if (expression bool > (var_ref b) (constant float (0.0)))
          ((if (expression bool > (var_ref c) (constant float (0.0))) ()
            (continue)))
index da1caad064e852963d9f22ba0d1cfc3c50dd26dd..8403bb2041b51edb851132882c96e4c8e5f8e9f2 100755 (executable)
@@ -8,6 +8,6 @@
 ((declare (out) float a)
  (function main
   (signature void (parameters)
-   ((loop ()
+   ((loop
      ((assign (x) (var_ref a) (constant float (1.000000))) continue))))))
 EOF
index df2b5483d862d4cf179d2bc794a8b9b2d1fd442a..98b74d71e76f0380b0273658cc641467c9250908 100644 (file)
@@ -1,5 +1,5 @@
 ((declare (out) float a)
  (function main
   (signature void (parameters)
-   ((loop ()
+   ((loop
      ((assign (x) (var_ref a) (constant float (1.000000)))))))))
index 0b46a7951301e257394a6d72fb77982af1a67ad5..1f62e73498d3e37f6e0b370b5e38dbefce5aa0cf 100755 (executable)
@@ -8,7 +8,7 @@
 ((declare (out) float a) (declare (out) float b)
  (function sub
   (signature float (parameters)
-   ((loop ()
+   ((loop
      ((assign (x) (var_ref a) (constant float (1.000000)))
       (return (constant float (2.000000)))))
     (assign (x) (var_ref b) (constant float (3.000000)))
index a1f3cdee713642c3c3ddc9ab8a56d77f0f90f52e..040d383b6040bda51e3a5e2b4faa7173dae5cdd2 100644 (file)
@@ -1,7 +1,7 @@
 ((declare (out) float a) (declare (out) float b)
  (function sub
   (signature float (parameters)
-   ((loop ()
+   ((loop
      ((assign (x) (var_ref a) (constant float (1.000000)))
       (return (constant float (2.000000)))))
     (assign (x) (var_ref b) (constant float (3.000000)))
index 0d73a6603b531552db7a6100ef1fa3a5f12e453c..42c4e7529b34baed9f60efe654b303040bb17cdd 100755 (executable)
@@ -8,7 +8,7 @@
 ((declare (out) float a) (declare (out) float b)
  (function sub
   (signature float (parameters)
-   ((loop ()
+   ((loop
      ((assign (x) (var_ref a) (constant float (1.000000)))
       (return (constant float (2.000000)))))
     (assign (x) (var_ref b) (constant float (3.000000)))
index 52bc99ccb3a79f2d047ff6986bd6d66e0cb963ef..792cbf69a99d5799b26756a878213253be1c5c9b 100644 (file)
@@ -6,7 +6,7 @@
     (declare (temporary) float return_value)
     (declare (temporary) bool return_flag)
     (assign (x) (var_ref return_flag) (constant bool (0)))
-    (loop ()
+    (loop
      ((assign (x) (var_ref a) (constant float (1.000000)))
       (assign (x) (var_ref return_value) (constant float (2.000000)))
       (assign (x) (var_ref return_flag) (constant bool (1)))
index 8b08c996b979b71066cb71da14279364bcf2035c..b3eef39b7e20fa2d9357e7e284fa243a506e04aa 100755 (executable)
@@ -8,7 +8,7 @@
 ((declare (out) float a) (declare (out) float b)
  (function sub
   (signature float (parameters)
-   ((loop ()
+   ((loop
      ((assign (x) (var_ref a) (constant float (1.000000)))
       (return (constant float (2.000000)))))
     (assign (x) (var_ref b) (constant float (3.000000)))
index 52bc99ccb3a79f2d047ff6986bd6d66e0cb963ef..792cbf69a99d5799b26756a878213253be1c5c9b 100644 (file)
@@ -6,7 +6,7 @@
     (declare (temporary) float return_value)
     (declare (temporary) bool return_flag)
     (assign (x) (var_ref return_flag) (constant bool (0)))
-    (loop ()
+    (loop
      ((assign (x) (var_ref a) (constant float (1.000000)))
       (assign (x) (var_ref return_value) (constant float (2.000000)))
       (assign (x) (var_ref return_flag) (constant bool (1)))
index f5b45bc058f332def9562fc7a11ee39e0a95f6e3..0408282d0d51233080e545e849adbefcf5b852d4 100755 (executable)
@@ -8,7 +8,7 @@
 ((declare (out) float a) (declare (out) float b)
  (function main
   (signature void (parameters)
-   ((loop ()
+   ((loop
      ((assign (x) (var_ref a) (constant float (1.000000))) (return)))
     (assign (x) (var_ref b) (constant float (2.000000)))))))
 EOF
index ae9359ff273358b71e0e6b5039754bc8d10cce56..569213e99f85de06e623891f73796bb5f81955a3 100644 (file)
@@ -1,6 +1,6 @@
 ((declare (out) float a) (declare (out) float b)
  (function main
   (signature void (parameters)
-   ((loop ()
+   ((loop
      ((assign (x) (var_ref a) (constant float (1.000000))) (return)))
     (assign (x) (var_ref b) (constant float (2.000000)))))))
index 2a02101d8ff87e970eb1b8fd23b523f896561547..a7e65c86cedc6111fb02456cc3ca009b216dc57d 100755 (executable)
@@ -8,7 +8,7 @@
 ((declare (out) float a) (declare (out) float b)
  (function main
   (signature void (parameters)
-   ((loop ()
+   ((loop
      ((assign (x) (var_ref a) (constant float (1.000000))) (return)))
     (assign (x) (var_ref b) (constant float (2.000000)))))))
 EOF
index d6dd1db68b50c1fee1d6702b649bbc4a23d5d6c5..66f3aeca59495316c254e19ad9449e7cf93200e6 100644 (file)
@@ -3,7 +3,7 @@
   (signature void (parameters)
    ((declare (temporary) bool return_flag)
     (assign (x) (var_ref return_flag) (constant bool (0)))
-    (loop ()
+    (loop
      ((assign (x) (var_ref a) (constant float (1.000000)))
       (assign (x) (var_ref return_flag) (constant bool (1)))
       break))
index 4c6f959122220922e15d0960b17805d97f8cb2e2..7a5efe5811e2bb527821bafdeca797def5e37523 100755 (executable)
@@ -8,7 +8,7 @@
 ((declare (out) float a) (declare (out) float b)
  (function main
   (signature void (parameters)
-   ((loop ()
+   ((loop
      ((assign (x) (var_ref a) (constant float (1.000000))) (return)))
     (assign (x) (var_ref b) (constant float (2.000000)))))))
 EOF
index d6dd1db68b50c1fee1d6702b649bbc4a23d5d6c5..66f3aeca59495316c254e19ad9449e7cf93200e6 100644 (file)
@@ -3,7 +3,7 @@
   (signature void (parameters)
    ((declare (temporary) bool return_flag)
     (assign (x) (var_ref return_flag) (constant bool (0)))
-    (loop ()
+    (loop
      ((assign (x) (var_ref a) (constant float (1.000000)))
       (assign (x) (var_ref return_flag) (constant bool (1)))
       break))
index 70eb997944283a766070c6691c040349e8558865..64ff8998f7e5005e223209745691f809a10a1914 100644 (file)
@@ -2181,11 +2181,6 @@ fs_visitor::visit(ir_if *ir)
 void
 fs_visitor::visit(ir_loop *ir)
 {
-   /* Any normative loop bounds should have been lowered by
-    * lower_bounded_loops().
-    */
-   assert(ir->normative_bound < 0);
-
    if (brw->gen < 6 && dispatch_width == 16) {
       fail("Can't support (non-uniform) control flow on 16-wide\n");
    }
index 12035c26793a1ec1aa82586724b059c4e507c86c..88aa169fc1fc98e2dc0b81fdf8d7b2c3fb2e967e 100644 (file)
@@ -213,8 +213,6 @@ brw_link_shader(struct gl_context *ctx, struct gl_shader_program *shProg)
           || progress;
       } while (progress);
 
-      lower_bounded_loops(shader->ir);
-
       /* Make a pass over the IR to add state references for any built-in
        * uniforms that are used.  This has to be done now (during linking).
        * Code generation doesn't happen until the first time this shader is
index d0e378b76ef4ae6d3992b19112e1b6cf49d19f85..81039ab0bc28c489d45c85577324eb962d68f097 100644 (file)
@@ -1007,11 +1007,6 @@ vec4_visitor::visit(ir_variable *ir)
 void
 vec4_visitor::visit(ir_loop *ir)
 {
-   /* Any normative loop bounds should have been lowered by
-    * lower_bounded_loops().
-    */
-   assert(ir->normative_bound < 0);
-
    /* We don't want debugging output to print the whole body of the
     * loop as the annotation.
     */
index 583cdef9f26c1b24f1c589c1c69727c4e4d1fcab..ad6c242e4d5f196e154e397046225534c448e050 100644 (file)
@@ -759,11 +759,6 @@ ir_to_mesa_visitor::visit(ir_variable *ir)
 void
 ir_to_mesa_visitor::visit(ir_loop *ir)
 {
-   /* Any normative loop bounds should have been lowered by
-    * lower_bounded_loops().
-    */
-   assert(ir->normative_bound < 0);
-
    emit(NULL, OPCODE_BGNLOOP);
 
    visit_exec_list(&ir->body_instructions, this);
@@ -3057,8 +3052,6 @@ _mesa_ir_link_shader(struct gl_context *ctx, struct gl_shader_program *prog)
          progress = lower_vector_insert(ir, true) || progress;
       } while (progress);
 
-      lower_bounded_loops(ir);
-
       validate_ir_tree(ir);
    }
 
index 18d2a5b0a41fd73e31cf0897b3e0cfe6bc6d4e30..a1ee24f66931bd0360fb6ba4ec20940da7ac2404 100644 (file)
@@ -1137,11 +1137,6 @@ glsl_to_tgsi_visitor::visit(ir_variable *ir)
 void
 glsl_to_tgsi_visitor::visit(ir_loop *ir)
 {
-   /* Any normative loop bounds should have been lowered by
-    * lower_bounded_loops().
-    */
-   assert(ir->normative_bound < 0);
-
    emit(NULL, TGSI_OPCODE_BGNLOOP);
 
    visit_exec_list(&ir->body_instructions, this);
@@ -5307,8 +5302,6 @@ st_link_shader(struct gl_context *ctx, struct gl_shader_program *prog)
 
       } while (progress);
 
-      lower_bounded_loops(ir);
-
       validate_ir_tree(ir);
    }