i965/vec4: Do separate copy followed by constant propagation after opt_vector_float().
authorMatt Turner <mattst88@gmail.com>
Sun, 21 Dec 2014 02:02:29 +0000 (18:02 -0800)
committerMatt Turner <mattst88@gmail.com>
Mon, 29 Dec 2014 18:08:18 +0000 (10:08 -0800)
total instructions in shared programs: 5877012 -> 5876617 (-0.01%)
instructions in affected programs:     33140 -> 32745 (-1.19%)

From before the commit that allows VF constant propagation (which hurt
some programs) to here, the results are:

total instructions in shared programs: 5877951 -> 5876617 (-0.02%)
instructions in affected programs:     123444 -> 122110 (-1.08%)

with no programs hurt.

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
src/mesa/drivers/dri/i965/brw_vec4.cpp

index 296223ccdd0878e2d2dfd5720769b54128a30b27..f389392636ffad8a5d6a6a33e591d385b740583b 100644 (file)
@@ -1800,7 +1800,8 @@ vec4_visitor::run()
 
    if (opt_vector_float()) {
       opt_cse();
-      opt_copy_propagation();
+      opt_copy_propagation(false);
+      opt_copy_propagation(true);
       dead_code_eliminate();
    }