On Monday, July 07, 2014 01:39:17 PM Matt Turner wrote: > ... on Gen7+. I'm not actually sure which class Gen6 fits into.
It looks like you actually did this for Gen6+, not Gen7+. I think that's
correct. Gen6 is when we moved from the mathbox shared function to an actual
MATH instruction.
Assuming you fix the commit message:
Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>
> ---
> src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp
b/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp
> index 10a8f6d..04ac242 100644
> --- a/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp
> +++ b/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp
> @@ -1356,6 +1356,7 @@
vec4_instruction_scheduler::issue_time(backend_instruction *inst)
> void
> instruction_scheduler::schedule_instructions(backend_instruction
*next_block_header)
> {
> + struct brw_context *brw = bv->brw;
> time = 0;
>
> /* Remove non-DAG heads from the list. */
> @@ -1424,7 +1425,7 @@
instruction_scheduler::schedule_instructions(backend_instruction
*next_block_hea
> * the next math instruction isn't going to make progress until the
first
> * is done.
> */
> - if (chosen->inst->is_math()) {
> + if (brw->gen < 6 && chosen->inst->is_math()) {
> foreach_in_list(schedule_node, n, &instructions) {
> if (n->inst->is_math())
> n->unblocked_time = MAX2(n->unblocked_time,
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
