AlexIndustrial/mesa
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

i965: Set brw->batch.emit only #ifdef DEBUG.

Browse Source 
It's only used inside #ifdef DEBUG. Cuts ~1.7k of .text, and more
importantly prevents a larger code size regression in the next commit
when the .used field is replaced and calculated on demand.

   text     data      bss      dec      hex  filename
4945468   195152    26192  5166812   4ed6dc  i965_dri.so before
4943740   195152    26192  5165084   4ed01c  i965_dri.so after

And surround the emit and total fields with #ifdef DEBUG to prevent
such mistakes from happening again.

Reviewed-by: Iago Toral Quiroga <itoral@igalia.com>
This commit is contained in:
Matt Turner
2015-07-07 18:51:30 -07:00
parent 0166b4c165
commit c04339486a
2 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/mesa/drivers/dri/i965/brw_context.h
+2
View File
@@ -870,7 +870,9 @@ struct intel_batchbuffer {
/** Last BO submitted to the hardware. Used for glFinish(). */ /** Last BO submitted to the hardware. Used for glFinish(). */
drm_intel_bo *last_bo; drm_intel_bo *last_bo;
#ifdef DEBUG
uint16_t emit, total; uint16_t emit, total;
#endif
uint16_t used, reserved_space; uint16_t used, reserved_space;
uint32_t *map; uint32_t *map;
uint32_t *cpu_map; uint32_t *cpu_map;
src/mesa/drivers/dri/i965/intel_batchbuffer.h
+1 -1
View File
@@ -138,8 +138,8 @@ intel_batchbuffer_begin(struct brw_context *brw, int n, enum brw_gpu_ring ring)
{ {
intel_batchbuffer_require_space(brw, n * 4, ring); intel_batchbuffer_require_space(brw, n * 4, ring);
brw->batch.emit = brw->batch.used;
#ifdef DEBUG #ifdef DEBUG
brw->batch.emit = brw->batch.used;
brw->batch.total = n; brw->batch.total = n;
#endif #endif
} }