diff options
| author | Kenneth Graunke <[email protected]> | 2012-01-05 13:54:41 -0800 |
|---|---|---|
| committer | Kenneth Graunke <[email protected]> | 2012-01-06 18:17:46 -0800 |
| commit | 7ccf04ebcf6284a68ab6c571dabc5f3e0c6b740b (patch) | |
| tree | 1e64fcec77b5bcb6f25c3f0358ec868fbb3af714 /src | |
| parent | de88e00c941d7d2d29ad5cbb87253ae116feab0e (diff) | |
i965/vs: Fix invalid array access in copy propagation.
Accessing virtual_grf_reg_map[inst->dst.reg] is invalid if
inst->dst.file != GRF. Since is_direct_copy already implies a GRF
destination, we can just move the check earlier.
Fixes a regression in commit 07ee9f374f2946f852896e9264c7fa83eafc3f16.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=44302
Signed-off-by: Kenneth Graunke <[email protected]>
Reviewed-by: Eric Anholt <[email protected]>
Diffstat (limited to 'src')
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp | 29 |
1 files changed, 15 insertions, 14 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp b/src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp index 08d8f5b7034..4d27c53ddf3 100644 --- a/src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp +++ b/src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp @@ -297,23 +297,24 @@ vec4_visitor::opt_copy_propagation() } /* Track available source registers. */ - const int reg = virtual_grf_reg_map[inst->dst.reg] + inst->dst.reg_offset; + if (inst->dst.file == GRF) { + const int reg = + virtual_grf_reg_map[inst->dst.reg] + inst->dst.reg_offset; - /* Update our destination's current channel values. For a direct copy, - * the value is the newly propagated source. Otherwise, we don't know - * the new value, so clear it. - */ - bool direct_copy = is_direct_copy(inst); - for (int i = 0; i < 4; i++) { - if (inst->dst.writemask & (1 << i)) { - cur_value[reg][i] = direct_copy ? &inst->src[0] : NULL; + /* Update our destination's current channel values. For a direct copy, + * the value is the newly propagated source. Otherwise, we don't know + * the new value, so clear it. + */ + bool direct_copy = is_direct_copy(inst); + for (int i = 0; i < 4; i++) { + if (inst->dst.writemask & (1 << i)) { + cur_value[reg][i] = direct_copy ? &inst->src[0] : NULL; + } } - } - /* Clear the records for any registers whose current value came from - * our destination's updated channels, as the two are no longer equal. - */ - if (inst->dst.file == GRF) { + /* Clear the records for any registers whose current value came from + * our destination's updated channels, as the two are no longer equal. + */ if (inst->dst.reladdr) memset(cur_value, 0, sizeof(cur_value)); else { |