diff options
| author | Kenneth Graunke <[email protected]> | 2016-04-29 21:12:15 -0700 |
|---|---|---|
| committer | Kenneth Graunke <[email protected]> | 2016-06-23 11:58:50 -0700 |
| commit | f7741c521119ce147215d94a4c238e84fc8b1130 (patch) | |
| tree | 3a8fc1bbc21f0742639b23423bf9af915bcc5347 | |
| parent | 586f4a42e78f171c28c3ae37adb8671e09bd74b4 (diff) | |
i965: Copy propagate before doing variable index lowering.
The scalar backend currently doesn't support variable indexing on
temporary arrays, but it does support it on uniform arrays, and
some stages support it for input arrays. Make sure these are
propagated through before exploding indirects into piles of
if-ladders unnecessarily.
On Broadwell, no instruction count change in shader-db.
total cycles in shared programs: 80675652 -> 80674928 (-0.00%)
cycles in affected programs: 649972 -> 649248 (-0.11%)
helped: 386
HURT: 165
This will help avoid code quality regressions in a future commit.
Cc: [email protected]
Signed-off-by: Kenneth Graunke <[email protected]>
Reviewed-by: Timothy Arceri <[email protected]>
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_link.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_link.cpp b/src/mesa/drivers/dri/i965/brw_link.cpp index 3bb04fa6eff..76c580b4648 100644 --- a/src/mesa/drivers/dri/i965/brw_link.cpp +++ b/src/mesa/drivers/dri/i965/brw_link.cpp @@ -130,6 +130,8 @@ process_glsl_ir(gl_shader_stage stage, lower_noise(shader->ir); lower_quadop_vector(shader->ir, false); + do_copy_propagation(shader->ir); + bool lowered_variable_indexing = lower_variable_index_to_cond_assign((gl_shader_stage)stage, shader->ir, |