mirror of
https://gitlab.freedesktop.org/mesa/mesa.git
synced 2025-12-21 22:20:14 +01:00
intel/compiler: Make vec4 generator take debug_enabled as a parameter
The callers already have this value, and we would like to make it follow different rules other than stage that might not be visible to the helper function, so just pass explicitly. Reviewed-by: Kenneth Graunke <kenneth@whitecape.org> Reviewed-by: Jordan Justen <jordan.l.justen@intel.com> Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/9779>
This commit is contained in:
parent
244d2daa00
commit
758eb18c6f
6 changed files with 15 additions and 13 deletions
|
|
@ -1435,7 +1435,7 @@ brw_compile_tes(const struct brw_compiler *compiler,
|
|||
assembly = brw_vec4_generate_assembly(compiler, log_data, mem_ctx, nir,
|
||||
&prog_data->base, v.cfg,
|
||||
v.performance_analysis.require(),
|
||||
stats);
|
||||
stats, debug_enabled);
|
||||
}
|
||||
|
||||
return assembly;
|
||||
|
|
|
|||
|
|
@ -2987,7 +2987,7 @@ brw_compile_vs(const struct brw_compiler *compiler,
|
|||
nir, &prog_data->base,
|
||||
v.cfg,
|
||||
v.performance_analysis.require(),
|
||||
params->stats);
|
||||
params->stats, debug_enabled);
|
||||
}
|
||||
|
||||
return assembly;
|
||||
|
|
|
|||
|
|
@ -49,7 +49,8 @@ brw_vec4_generate_assembly(const struct brw_compiler *compiler,
|
|||
struct brw_vue_prog_data *prog_data,
|
||||
const struct cfg_t *cfg,
|
||||
const brw::performance &perf,
|
||||
struct brw_compile_stats *stats);
|
||||
struct brw_compile_stats *stats,
|
||||
bool debug_enabled);
|
||||
|
||||
#ifdef __cplusplus
|
||||
} /* extern "C" */
|
||||
|
|
|
|||
|
|
@ -1473,12 +1473,11 @@ generate_code(struct brw_codegen *p,
|
|||
struct brw_vue_prog_data *prog_data,
|
||||
const struct cfg_t *cfg,
|
||||
const performance &perf,
|
||||
struct brw_compile_stats *stats)
|
||||
struct brw_compile_stats *stats,
|
||||
bool debug_enabled)
|
||||
{
|
||||
const struct gen_device_info *devinfo = p->devinfo;
|
||||
const char *stage_abbrev = _mesa_shader_stage_to_abbrev(nir->info.stage);
|
||||
bool debug_flag = INTEL_DEBUG &
|
||||
intel_debug_flag_for_shader_stage(nir->info.stage);
|
||||
struct disasm_info *disasm_info = disasm_initialize(devinfo, cfg);
|
||||
|
||||
/* `send_count` explicitly does not include spills or fills, as we'd
|
||||
|
|
@ -1493,7 +1492,7 @@ generate_code(struct brw_codegen *p,
|
|||
foreach_block_and_inst (block, vec4_instruction, inst, cfg) {
|
||||
struct brw_reg src[3], dst;
|
||||
|
||||
if (unlikely(debug_flag))
|
||||
if (unlikely(debug_enabled))
|
||||
disasm_annotate(disasm_info, inst, p->next_insn_offset);
|
||||
|
||||
for (unsigned int i = 0; i < 3; i++) {
|
||||
|
|
@ -2180,7 +2179,7 @@ generate_code(struct brw_codegen *p,
|
|||
brw_compact_instructions(p, 0, disasm_info);
|
||||
int after_size = p->next_insn_offset;
|
||||
|
||||
if (unlikely(debug_flag)) {
|
||||
if (unlikely(debug_enabled)) {
|
||||
unsigned char sha1[21];
|
||||
char sha1buf[41];
|
||||
|
||||
|
|
@ -2235,13 +2234,15 @@ brw_vec4_generate_assembly(const struct brw_compiler *compiler,
|
|||
struct brw_vue_prog_data *prog_data,
|
||||
const struct cfg_t *cfg,
|
||||
const performance &perf,
|
||||
struct brw_compile_stats *stats)
|
||||
struct brw_compile_stats *stats,
|
||||
bool debug_enabled)
|
||||
{
|
||||
struct brw_codegen *p = rzalloc(mem_ctx, struct brw_codegen);
|
||||
brw_init_codegen(compiler->devinfo, p, mem_ctx);
|
||||
brw_set_default_access_mode(p, BRW_ALIGN_16);
|
||||
|
||||
generate_code(p, compiler, log_data, nir, prog_data, cfg, perf, stats);
|
||||
generate_code(p, compiler, log_data, nir, prog_data, cfg, perf, stats,
|
||||
debug_enabled);
|
||||
|
||||
assert(prog_data->base.const_data_size == 0);
|
||||
if (nir->constant_data_size > 0) {
|
||||
|
|
|
|||
|
|
@ -875,7 +875,7 @@ brw_compile_gs(const struct brw_compiler *compiler, void *log_data,
|
|||
nir, &prog_data->base,
|
||||
v.cfg,
|
||||
v.performance_analysis.require(),
|
||||
stats);
|
||||
stats, debug_enabled);
|
||||
} else {
|
||||
/* These variables could be modified by the execution of the GS
|
||||
* visitor if it packed the uniforms in the push constant buffer.
|
||||
|
|
@ -940,7 +940,7 @@ brw_compile_gs(const struct brw_compiler *compiler, void *log_data,
|
|||
ret = brw_vec4_generate_assembly(compiler, log_data, mem_ctx, nir,
|
||||
&prog_data->base, gs->cfg,
|
||||
gs->performance_analysis.require(),
|
||||
stats);
|
||||
stats, debug_enabled);
|
||||
}
|
||||
|
||||
delete gs;
|
||||
|
|
|
|||
|
|
@ -500,7 +500,7 @@ brw_compile_tcs(const struct brw_compiler *compiler,
|
|||
assembly = brw_vec4_generate_assembly(compiler, log_data, mem_ctx, nir,
|
||||
&prog_data->base, v.cfg,
|
||||
v.performance_analysis.require(),
|
||||
stats);
|
||||
stats, debug_enabled);
|
||||
}
|
||||
|
||||
return assembly;
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue