glsl: be more consistent about printing constants
authorConnor Abbott <cwabbott0@gmail.com>
Wed, 28 May 2014 01:23:02 +0000 (21:23 -0400)
committerKenneth Graunke <kenneth@whitecape.org>
Wed, 28 May 2014 22:05:59 +0000 (15:05 -0700)
Make sure that we print the same number of digits when printing 0.0 as
any other floating-point number. This will make generating expected
output files for tests easier. To avoid breaking "make check," update
the generated tests for lower_jumps before the next commit which will
bring create_test_cases.py in line with them.

Signed-off-by: Connor Abbott <cwabbott0@gmail.com>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Matt Turner <mattst88@gmail.com>
23 files changed:
src/glsl/ir_print_visitor.cpp
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.expected
src/glsl/tests/lower_jumps/lower_breaks_3.opt_test.expected
src/glsl/tests/lower_jumps/lower_breaks_4.opt_test.expected
src/glsl/tests/lower_jumps/lower_breaks_5.opt_test.expected
src/glsl/tests/lower_jumps/lower_breaks_6.opt_test.expected
src/glsl/tests/lower_jumps/lower_guarded_conditional_break.opt_test.expected
src/glsl/tests/lower_jumps/lower_pulled_out_jump.opt_test.expected
src/glsl/tests/lower_jumps/lower_returns_3.opt_test.expected
src/glsl/tests/lower_jumps/lower_returns_4.opt_test.expected
src/glsl/tests/lower_jumps/lower_returns_main_false.opt_test.expected
src/glsl/tests/lower_jumps/lower_returns_main_true.opt_test.expected
src/glsl/tests/lower_jumps/lower_returns_sub_false.opt_test.expected
src/glsl/tests/lower_jumps/lower_returns_sub_true.opt_test.expected
src/glsl/tests/lower_jumps/lower_unified_returns.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_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_and_break.opt_test

index 418f0515153417dda243ea630794b8161f001640..c4a6f9c9f1c2c5c87cb75e8697cde561399a669f 100644 (file)
@@ -430,7 +430,7 @@ void ir_print_visitor::visit(ir_constant *ir)
         case GLSL_TYPE_FLOAT:
             if (ir->value.f[i] == 0.0f)
                /* 0.0 == -0.0, so print with %f to get the proper sign. */
-               fprintf(f, "%.1f", ir->value.f[i]);
+               fprintf(f, "%f", ir->value.f[i]);
             else if (fabs(ir->value.f[i]) < 0.000001f)
                fprintf(f, "%a", ir->value.f[i]);
             else if (fabs(ir->value.f[i]) > 1000000.0f)
index b412ba8f29f5f82c5bde9e0d820277ff2bb08a4f..e2d4ed1f343411ed489117d3974331dc2bc1025d 100755 (executable)
@@ -8,6 +8,5 @@
 ((declare (out) float a)
  (function main
   (signature void (parameters)
-   ((loop
-     ((assign (x) (var_ref a) (constant float (1.000000))) break))))))
+   ((loop ((assign (x) (var_ref a) (constant float (1.000000))) break))))))
 EOF
index 56ef3e4246537d44cb74308bd299ea6901da2d67..270a43d905897f234ee8c42d8a85f2b53f04ec5c 100644 (file)
@@ -1,5 +1,4 @@
 ((declare (out) float a)
  (function main
   (signature void (parameters)
-   ((loop
-     ((assign (x) (var_ref a) (constant float (1.000000))) break))))))
+   ((loop ((assign (x) (var_ref a) (constant float (1.000000))) break))))))
index dc231f9753b689a0185fe7380bbd993758d62dbc..73a1d56fa5f486fe04783cfa9bc105f99a4125f7 100644 (file)
@@ -3,5 +3,5 @@
   (signature void (parameters)
    ((loop
      ((assign (x) (var_ref a) (constant float (1.000000)))
-      (if (expression bool > (var_ref b) (constant float (0.0))) (break)
+      (if (expression bool > (var_ref b) (constant float (0.000000))) (break)
        ())))))))
index 8131b66ff4b03fa374f087283eca8d2ac22a66db..53d5392d05e89d821531b59e6e37755166be620c 100644 (file)
@@ -3,6 +3,6 @@
   (signature void (parameters)
    ((loop
      ((assign (x) (var_ref a) (constant float (1.000000)))
-      (if (expression bool > (var_ref b) (constant float (0.0)))
+      (if (expression bool > (var_ref b) (constant float (0.000000)))
        ((assign (x) (var_ref c) (constant float (1.000000))) break)
        ())))))))
index 94dcb37be04eaa6e2c835fc4f5ad73dcf02635c8..ecf28c4e5ab8af5efa2358052d3020e656b3de20 100644 (file)
@@ -3,5 +3,5 @@
   (signature void (parameters)
    ((loop
      ((assign (x) (var_ref a) (constant float (1.000000)))
-      (if (expression bool > (var_ref b) (constant float (0.0))) ()
+      (if (expression bool > (var_ref b) (constant float (0.000000))) ()
        (break))))))))
index 5b46ccb677331d26610c9a7207c51370bbf9dec3..418f8798cf4389abfe81cf578f68c65507521fb8 100644 (file)
@@ -3,5 +3,5 @@
   (signature void (parameters)
    ((loop
      ((assign (x) (var_ref a) (constant float (1.000000)))
-      (if (expression bool > (var_ref b) (constant float (0.0))) ()
+      (if (expression bool > (var_ref b) (constant float (0.000000))) ()
        ((assign (x) (var_ref c) (constant float (1.000000))) break))))))))
index 967ce642a4b2d8a291fb21e64d10d3980ec139b9..5582d08b2366cd34c16ba79a58b6e95f084793ad 100644 (file)
@@ -8,15 +8,15 @@
     (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)))
-       ((if (expression bool > (var_ref ba) (constant float (0.0)))
-         ((if (expression bool > (var_ref bb) (constant float (0.0)))
+      (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)))
            ((assign (x) (var_ref execute_flag) (constant bool (0))))
            ()))
          ())
         (if (var_ref execute_flag)
-         ((if (expression bool > (var_ref ca) (constant float (0.0)))
-           ((if (expression bool > (var_ref cb) (constant float (0.0)))
+         ((if (expression bool > (var_ref ca) (constant float (0.000000)))
+           ((if (expression bool > (var_ref cb) (constant float (0.000000)))
              ((assign (x) (var_ref break_flag) (constant bool (1)))
               (assign (x) (var_ref execute_flag) (constant bool (0))))
              ()))
index 841073ed909de7d9b1fc3319f59c8c889ce22e40..037920668e96b346fb8ffe78f7404d06572f30ac 100644 (file)
@@ -6,13 +6,13 @@
     (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)))
-       ((if (expression bool > (var_ref ab) (constant float (0.0)))
+      (if (expression bool > (var_ref aa) (constant float (0.000000)))
+       ((if (expression bool > (var_ref ab) (constant float (0.000000)))
          ((assign (x) (var_ref execute_flag) (constant bool (0))))
          ()))
        ())
       (if (var_ref execute_flag)
-       ((if (expression bool > (var_ref b) (constant float (0.0)))
+       ((if (expression bool > (var_ref b) (constant float (0.000000)))
          ((assign (x) (var_ref break_flag) (constant bool (1)))
           (assign (x) (var_ref execute_flag) (constant bool (0))))
          ()))
index cf2ef3f080a67aeb83acf147a824f56087feda50..47e8c795c456d0ffcbbb2d137ae2c061d1784ef0 100644 (file)
@@ -7,16 +7,16 @@
     (assign (x) (var_ref execute_flag) (constant bool (1)))
     (declare (temporary) bool return_flag)
     (assign (x) (var_ref return_flag) (constant bool (0)))
-    (if (expression bool > (var_ref aa) (constant float (0.0)))
-     ((if (expression bool > (var_ref ab) (constant float (0.0)))
+    (if (expression bool > (var_ref aa) (constant float (0.000000)))
+     ((if (expression bool > (var_ref ab) (constant float (0.000000)))
        ((assign (x) (var_ref return_flag) (constant bool (1)))
         (assign (x) (var_ref execute_flag) (constant bool (0))))
        ()))
      ())
     (if (var_ref execute_flag)
      ((loop
-       ((if (expression bool > (var_ref b) (constant float (0.0)))
-         ((if (expression bool > (var_ref c) (constant float (0.0))) ()
+       ((if (expression bool > (var_ref b) (constant float (0.000000)))
+         ((if (expression bool > (var_ref c) (constant float (0.000000))) ()
            (continue)))
          ((assign (x) (var_ref return_flag) (constant bool (1)))))
         break))
index 5b62bbcde99f8a9ef6fccb8199120c3cdfc225e1..d4835e96b7c92ea40405b1acd654a1171ccd38f9 100644 (file)
@@ -6,8 +6,8 @@
     (declare (temporary) float return_value)
     (declare (temporary) bool return_flag)
     (assign (x) (var_ref return_flag) (constant bool (0)))
-    (if (expression bool > (var_ref a) (constant float (0.0)))
-     ((if (expression bool > (var_ref b) (constant float (0.0)))
+    (if (expression bool > (var_ref a) (constant float (0.000000)))
+     ((if (expression bool > (var_ref b) (constant float (0.000000)))
        ((assign (x) (var_ref return_value) (constant float (1.000000)))
         (assign (x) (var_ref return_flag) (constant bool (1)))
         (assign (x) (var_ref execute_flag) (constant bool (0))))
index 07c6842d20c7dccad5e04e7c4f8e50010d4aca33..b551a066f4362341a67cd53301064ecc9bb57a91 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)))
-    (if (expression bool > (var_ref a) (constant float (0.0)))
+    (if (expression bool > (var_ref a) (constant float (0.000000)))
      ((assign (x) (var_ref return_value) (constant float (1.000000)))
       (assign (x) (var_ref return_flag) (constant bool (1)))
       (assign (x) (var_ref execute_flag) (constant bool (0))))
index 7e3fe314f96807823a4ae1e0a995679be659f18c..e8b36f14478791ef9638ee40ac884859fa4c8dea 100644 (file)
@@ -1,8 +1,8 @@
 ((declare (in) float a) (declare (in) float b)
  (function main
   (signature void (parameters)
-   ((if (expression bool > (var_ref a) (constant float (0.0)))
-     ((if (expression bool > (var_ref b) (constant float (0.0)))
+   ((if (expression bool > (var_ref a) (constant float (0.000000)))
+     ((if (expression bool > (var_ref b) (constant float (0.000000)))
        ((return))
        ()))
      ())))))
index b47f5a433dec27894265e5f0a0fedb0a6d4a5d88..e15a97d1db281bf692b3dc3317d0049546b615b8 100644 (file)
@@ -5,8 +5,8 @@
     (assign (x) (var_ref execute_flag) (constant bool (1)))
     (declare (temporary) bool return_flag)
     (assign (x) (var_ref return_flag) (constant bool (0)))
-    (if (expression bool > (var_ref a) (constant float (0.0)))
-     ((if (expression bool > (var_ref b) (constant float (0.0)))
+    (if (expression bool > (var_ref a) (constant float (0.000000)))
+     ((if (expression bool > (var_ref b) (constant float (0.000000)))
        ((assign (x) (var_ref return_flag) (constant bool (1)))
         (assign (x) (var_ref execute_flag) (constant bool (0))))
        ()))
index 7424968b248eb41c08b03f858ce77fc39a76c5fa..07db6e708f490344fd389ddada8b8ed8edb33d86 100644 (file)
@@ -1,8 +1,8 @@
 ((declare (in) float a) (declare (in) float b)
  (function sub
   (signature void (parameters)
-   ((if (expression bool > (var_ref a) (constant float (0.0)))
-     ((if (expression bool > (var_ref b) (constant float (0.0)))
+   ((if (expression bool > (var_ref a) (constant float (0.000000)))
+     ((if (expression bool > (var_ref b) (constant float (0.000000)))
        ((return))
        ()))
      ())))))
index 1a3eae5da7616fa8d36fceb81ec0df50e659faf2..311098023514a7d36b598325fbb251a949ec8e79 100644 (file)
@@ -5,8 +5,8 @@
     (assign (x) (var_ref execute_flag) (constant bool (1)))
     (declare (temporary) bool return_flag)
     (assign (x) (var_ref return_flag) (constant bool (0)))
-    (if (expression bool > (var_ref a) (constant float (0.0)))
-     ((if (expression bool > (var_ref b) (constant float (0.0)))
+    (if (expression bool > (var_ref a) (constant float (0.000000)))
+     ((if (expression bool > (var_ref b) (constant float (0.000000)))
        ((assign (x) (var_ref return_flag) (constant bool (1)))
         (assign (x) (var_ref execute_flag) (constant bool (0))))
        ()))
index c0b51e1baaa4a39f312efe172835c3fada485175..271cd3b462ec7046f212a98d0f782a110f91efc6 100644 (file)
@@ -6,15 +6,15 @@
     (assign (x) (var_ref execute_flag) (constant bool (1)))
     (declare (temporary) bool return_flag)
     (assign (x) (var_ref return_flag) (constant bool (0)))
-    (if (expression bool > (var_ref aa) (constant float (0.0)))
-     ((if (expression bool > (var_ref ab) (constant float (0.0)))
+    (if (expression bool > (var_ref aa) (constant float (0.000000)))
+     ((if (expression bool > (var_ref ab) (constant float (0.000000)))
        ((assign (x) (var_ref return_flag) (constant bool (1)))
         (assign (x) (var_ref execute_flag) (constant bool (0))))
        ()))
      ())
     (if (var_ref execute_flag)
-     ((if (expression bool > (var_ref b) (constant float (0.0)))
-       ((if (expression bool > (var_ref c) (constant float (0.0))) () ())
+     ((if (expression bool > (var_ref b) (constant float (0.000000)))
+       ((if (expression bool > (var_ref c) (constant float (0.000000))) () ())
         (assign (x) (var_ref return_flag) (constant bool (1)))
         (assign (x) (var_ref execute_flag) (constant bool (0))))
        ()))
index 8403bb2041b51edb851132882c96e4c8e5f8e9f2..5fa1134f985e87e639b12b14af36fe4c8944194f 100755 (executable)
@@ -8,6 +8,5 @@
 ((declare (out) float a)
  (function main
   (signature void (parameters)
-   ((loop
-     ((assign (x) (var_ref a) (constant float (1.000000))) continue))))))
+   ((loop ((assign (x) (var_ref a) (constant float (1.000000))) continue))))))
 EOF
index 98b74d71e76f0380b0273658cc641467c9250908..2a7ec2ecbecc40b2982a5e849da0034bc1ede6b7 100644 (file)
@@ -1,5 +1,4 @@
 ((declare (out) float a)
  (function main
   (signature void (parameters)
-   ((loop
-     ((assign (x) (var_ref a) (constant float (1.000000)))))))))
+   ((loop ((assign (x) (var_ref a) (constant float (1.000000)))))))))
index 0408282d0d51233080e545e849adbefcf5b852d4..79442433c08f8d07981070d9ac6fcba3f2fb72a8 100755 (executable)
@@ -8,7 +8,6 @@
 ((declare (out) float a) (declare (out) float b)
  (function main
   (signature void (parameters)
-   ((loop
-     ((assign (x) (var_ref a) (constant float (1.000000))) (return)))
+   ((loop ((assign (x) (var_ref a) (constant float (1.000000))) (return)))
     (assign (x) (var_ref b) (constant float (2.000000)))))))
 EOF
index 569213e99f85de06e623891f73796bb5f81955a3..156e95edf7d0228e2d8530e8156f57346d327c84 100644 (file)
@@ -1,6 +1,5 @@
 ((declare (out) float a) (declare (out) float b)
  (function main
   (signature void (parameters)
-   ((loop
-     ((assign (x) (var_ref a) (constant float (1.000000))) (return)))
+   ((loop ((assign (x) (var_ref a) (constant float (1.000000))) (return)))
     (assign (x) (var_ref b) (constant float (2.000000)))))))
index a7e65c86cedc6111fb02456cc3ca009b216dc57d..76469f73d6af79b823f8b4768a8f68b59477ec53 100755 (executable)
@@ -8,7 +8,6 @@
 ((declare (out) float a) (declare (out) float b)
  (function main
   (signature void (parameters)
-   ((loop
-     ((assign (x) (var_ref a) (constant float (1.000000))) (return)))
+   ((loop ((assign (x) (var_ref a) (constant float (1.000000))) (return)))
     (assign (x) (var_ref b) (constant float (2.000000)))))))
 EOF
index 7a5efe5811e2bb527821bafdeca797def5e37523..1d145b4fcb77bd32b2abfa8560e100069c346297 100755 (executable)
@@ -8,7 +8,6 @@
 ((declare (out) float a) (declare (out) float b)
  (function main
   (signature void (parameters)
-   ((loop
-     ((assign (x) (var_ref a) (constant float (1.000000))) (return)))
+   ((loop ((assign (x) (var_ref a) (constant float (1.000000))) (return)))
     (assign (x) (var_ref b) (constant float (2.000000)))))))
 EOF