[Mesa-dev] [PATCH 01/10] i965: Remove incorrect cycle estimates.

[Francisco Jerez]

Matt Turner <mattst88 at gmail.com> writes:

> These printed the cycle count the last basic block (sched.time is set
> per basic block!). We have accurate, full program, data printed
> elsewhere.

Reviewed-by: Francisco Jerez <currojerez at riseup.net>

> ---
>  src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp | 10 ----------
>  1 file changed, 10 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp b/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp
> index 4f97577..2c7e4f7 100644
> --- a/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp
> +++ b/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp
> @@ -1684,11 +1684,6 @@ fs_visitor::schedule_instructions(instruction_scheduler_mode mode)
>                                    cfg->num_blocks, mode);
>     sched.run(cfg);
>  
> -   if (unlikely(debug_enabled) && mode == SCHEDULE_POST) {
> -      fprintf(stderr, "%s%d estimated execution time: %d cycles\n",
> -              stage_abbrev, dispatch_width, sched.time);
> -   }
> -
>     invalidate_live_intervals();
>  }
>  
> @@ -1698,10 +1693,5 @@ vec4_visitor::opt_schedule_instructions()
>     vec4_instruction_scheduler sched(this, prog_data->total_grf);
>     sched.run(cfg);
>  
> -   if (unlikely(debug_enabled)) {
> -      fprintf(stderr, "%s estimated execution time: %d cycles\n",
> -              stage_abbrev, sched.time);
> -   }
> -
>     invalidate_live_intervals();
>  }
> -- 
> 2.4.10
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 212 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20160315/0885f1ba/attachment.sig>