fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-05-01 10:18:05 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

i965: Tidy shader time printing code by using printf's field widths.

Browse source 
We can use %-6s%-6s rather than manually counting characters, resulting
in much more readable code.

This necessitates a small secondary change: using "total fs16" and ""
now causes the "" string to be padded out to 6 characters, resulting in
too much whitespace.  Splitting it into "total" and "fs16" produces the
same output as before.

Reviewed-by: Eric Anholt <eric@anholt.net>
Signed-off-by: Kenneth Graunke <kenneth@whitecape.org>
This commit is contained in:
Kenneth Graunke 2013-03-28 00:18:46 -07:00 • committed by Eric Anholt
parent 6192e9b377
commit 9fe47756b3
1 changed files with 4 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
src/mesa/drivers/dri/i965/brw_program.c
View file

@ -280,15 +280,7 @@ static void
print_shader_time_line(const char *stage, const char *name,
int shader_num, uint64_t time, uint64_t total)
{
int namelen = printf("%s ", stage);
while (namelen++ < 6)
printf(" ");
namelen += printf("%s ", name);
while (namelen++ < 13)
printf(" ");
printf("%-6s%-6s", stage, name);
if (shader_num != -1)
printf("%4d: ", shader_num);
@ -429,9 +421,9 @@ brw_report_shader_time(struct brw_context *brw)
}
printf("\n");
print_shader_time_line("total vs", "", -1, total_by_type[ST_VS], total);
print_shader_time_line("total fs8", "", -1, total_by_type[ST_FS8], total);
print_shader_time_line("total fs16", "", -1, total_by_type[ST_FS16], total);
print_shader_time_line("total", "vs", -1, total_by_type[ST_VS], total);
print_shader_time_line("total", "fs8", -1, total_by_type[ST_FS8], total);
print_shader_time_line("total", "fs16", -1, total_by_type[ST_FS16], total);
}
static void