Mesa (master): i965/fs: Call calculate_cfg() before optimize().

Matt Turner mattst88 at kemper.freedesktop.org
Sun Feb 15 20:24:52 UTC 2015


Module: Mesa
Branch: master
Commit: fa124a337ca10d2c5d2d81a89dc8c21a7ba2f58b
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=fa124a337ca10d2c5d2d81a89dc8c21a7ba2f58b

Author: Matt Turner <mattst88 at gmail.com>
Date:   Fri Feb 13 10:34:39 2015 -0800

i965/fs: Call calculate_cfg() before optimize().

The CFG is fundamental to the FS IR, not merely a piece of optimization.

Reviewed-by: Ben Widawsky <ben at bwidawsk.net>

---

 src/mesa/drivers/dri/i965/brw_fs.cpp |    6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp b/src/mesa/drivers/dri/i965/brw_fs.cpp
index 0f1300c..a311266 100644
--- a/src/mesa/drivers/dri/i965/brw_fs.cpp
+++ b/src/mesa/drivers/dri/i965/brw_fs.cpp
@@ -3557,8 +3557,6 @@ fs_visitor::optimize()
 {
    const char *stage_name = stage == MESA_SHADER_VERTEX ? "vs" : "fs";
 
-   calculate_cfg();
-
    split_virtual_grfs();
 
    move_uniform_array_access_to_pull_constants();
@@ -3734,6 +3732,8 @@ fs_visitor::run_vs()
 
    emit_urb_writes();
 
+   calculate_cfg();
+
    optimize();
 
    assign_curb_setup();
@@ -3813,6 +3813,8 @@ fs_visitor::run_fs()
 
       emit_fb_writes();
 
+      calculate_cfg();
+
       optimize();
 
       assign_curb_setup();




More information about the mesa-commit mailing list