From c052087038c80c3ab568cec9159ed931a94d6574 Mon Sep 17 00:00:00 2001 From: Rob Clark Date: Sat, 13 Jun 2020 10:49:47 -0700 Subject: [PATCH] freedreno/ir3: re-work assembler API Just pass thru the variant, since it has everything we need. And will be needed in the next patch. Signed-off-by: Rob Clark Part-of: --- src/freedreno/computerator/ir3_asm.c | 2 +- src/freedreno/ir3/ir3.c | 21 ++++++++++++------- src/freedreno/ir3/ir3.h | 7 ++++--- src/freedreno/ir3/ir3_shader.c | 10 ++++----- src/freedreno/ir3/ir3_shader.h | 2 +- .../drivers/freedreno/ir3/ir3_cmdline.c | 2 +- 6 files changed, 24 insertions(+), 20 deletions(-) diff --git a/src/freedreno/computerator/ir3_asm.c b/src/freedreno/computerator/ir3_asm.c index cbab6e8e16f..b68cd37a680 100644 --- a/src/freedreno/computerator/ir3_asm.c +++ b/src/freedreno/computerator/ir3_asm.c @@ -53,7 +53,7 @@ ir3_asm_assemble(struct ir3_compiler *c, FILE *in) kernel->base.num_bufs = kernel->info.num_bufs; memcpy(kernel->base.buf_sizes, kernel->info.buf_sizes, sizeof(kernel->base.buf_sizes)); - kernel->bin = ir3_shader_assemble(v, c->gpu_id); + kernel->bin = ir3_shader_assemble(v); unsigned sz = v->info.sizedwords * 4; diff --git a/src/freedreno/ir3/ir3.c b/src/freedreno/ir3/ir3.c index 209dfb36b25..1751ab8a6c9 100644 --- a/src/freedreno/ir3/ir3.c +++ b/src/freedreno/ir3/ir3.c @@ -79,6 +79,8 @@ void ir3_destroy(struct ir3 *shader) static uint32_t reg(struct ir3_register *reg, struct ir3_info *info, uint32_t repeat, uint32_t valid_flags) { + struct ir3_shader_variant *v = info->data; + bool mergedregs = v->shader->compiler->gpu_id >= 600; reg_t val = { .dummy32 = 0 }; if (reg->flags & ~valid_flags) { @@ -112,7 +114,7 @@ static uint32_t reg(struct ir3_register *reg, struct ir3_info *info, /* ignore writes to dummy register r63.x */ } else if (max < regid(48, 0)) { if (reg->flags & IR3_REG_HALF) { - if (info->gpu_id >= 600) { + if (mergedregs) { /* starting w/ a6xx, half regs conflict with full regs: */ info->max_reg = MAX2(info->max_reg, max >> 3); } else { @@ -130,11 +132,12 @@ static uint32_t reg(struct ir3_register *reg, struct ir3_info *info, static int emit_cat0(struct ir3_instruction *instr, void *ptr, struct ir3_info *info) { + struct ir3_shader_variant *v = info->data; instr_cat0_t *cat0 = ptr; - if (info->gpu_id >= 500) { + if (v->shader->compiler->gpu_id >= 500) { cat0->a5xx.immed = instr->cat0.immed; - } else if (info->gpu_id >= 400) { + } else if (v->shader->compiler->gpu_id >= 400) { cat0->a4xx.immed = instr->cat0.immed; } else { cat0->a3xx.immed = instr->cat0.immed; @@ -628,13 +631,14 @@ static int emit_cat6_a6xx(struct ir3_instruction *instr, void *ptr, static int emit_cat6(struct ir3_instruction *instr, void *ptr, struct ir3_info *info) { + struct ir3_shader_variant *v = info->data; struct ir3_register *dst, *src1, *src2; instr_cat6_t *cat6 = ptr; /* In a6xx we start using a new instruction encoding for some of * these instructions: */ - if (info->gpu_id >= 600) { + if (v->shader->compiler->gpu_id >= 600) { switch (instr->opc) { case OPC_ATOMIC_ADD: case OPC_ATOMIC_SUB: @@ -912,13 +916,14 @@ static int (*emit[])(struct ir3_instruction *instr, void *ptr, emit_cat7, }; -void * ir3_assemble(struct ir3 *shader, struct ir3_info *info, - uint32_t gpu_id) +void * ir3_assemble(struct ir3_shader_variant *v) { uint32_t *ptr, *dwords; + struct ir3_info *info = &v->info; + struct ir3 *shader = v->ir; memset(info, 0, sizeof(*info)); - info->gpu_id = gpu_id; + info->data = v; info->max_reg = -1; info->max_half_reg = -1; info->max_const = -1; @@ -933,7 +938,7 @@ void * ir3_assemble(struct ir3 *shader, struct ir3_info *info, * instructions on a4xx or sets of 4 instructions on a3xx), * so pad out w/ NOPs if needed: (NOTE each instruction is 64bits) */ - if (gpu_id >= 400) { + if (v->shader->compiler->gpu_id >= 400) { info->sizedwords = align(info->sizedwords, 16 * 2); } else { info->sizedwords = align(info->sizedwords, 4 * 2); diff --git a/src/freedreno/ir3/ir3.h b/src/freedreno/ir3/ir3.h index c0d46eea8f5..48e340d09cd 100644 --- a/src/freedreno/ir3/ir3.h +++ b/src/freedreno/ir3/ir3.h @@ -44,7 +44,7 @@ struct ir3_instruction; struct ir3_block; struct ir3_info { - uint32_t gpu_id; + void *data; /* used internally in ir3 assembler */ uint16_t sizedwords; uint16_t instrs_count; /* expanded to account for rpt's */ uint16_t nops_count; /* # of nop instructions, including nopN */ @@ -556,8 +556,9 @@ block_id(struct ir3_block *block) struct ir3 * ir3_create(struct ir3_compiler *compiler, gl_shader_stage type); void ir3_destroy(struct ir3 *shader); -void * ir3_assemble(struct ir3 *shader, - struct ir3_info *info, uint32_t gpu_id); + +struct ir3_shader_variant; +void * ir3_assemble(struct ir3_shader_variant *v); void * ir3_alloc(struct ir3 *shader, int sz); struct ir3_block * ir3_block_create(struct ir3 *shader); diff --git a/src/freedreno/ir3/ir3_shader.c b/src/freedreno/ir3/ir3_shader.c index c0f33263e88..a2ca295845e 100644 --- a/src/freedreno/ir3/ir3_shader.c +++ b/src/freedreno/ir3/ir3_shader.c @@ -131,11 +131,12 @@ fixup_regfootprint(struct ir3_shader_variant *v, uint32_t gpu_id) /* wrapper for ir3_assemble() which does some info fixup based on * shader state. Non-static since used by ir3_cmdline too. */ -void * ir3_shader_assemble(struct ir3_shader_variant *v, uint32_t gpu_id) +void * ir3_shader_assemble(struct ir3_shader_variant *v) { + unsigned gpu_id = v->shader->compiler->gpu_id; void *bin; - bin = ir3_assemble(v->ir, &v->info, gpu_id); + bin = ir3_assemble(v); if (!bin) return NULL; @@ -159,10 +160,7 @@ void * ir3_shader_assemble(struct ir3_shader_variant *v, uint32_t gpu_id) static void assemble_variant(struct ir3_shader_variant *v) { - struct ir3_compiler *compiler = v->shader->compiler; - uint32_t gpu_id = compiler->gpu_id; - - v->bin = ir3_shader_assemble(v, gpu_id); + v->bin = ir3_shader_assemble(v); if (shader_debug_enabled(v->shader->type)) { fprintf(stdout, "Native code for unnamed %s shader %s:\n", diff --git a/src/freedreno/ir3/ir3_shader.h b/src/freedreno/ir3/ir3_shader.h index 2fe79ea7a0f..472e4ab265b 100644 --- a/src/freedreno/ir3/ir3_shader.h +++ b/src/freedreno/ir3/ir3_shader.h @@ -642,7 +642,7 @@ struct ir3_shader { struct ir3_shader_key key_mask; }; -void * ir3_shader_assemble(struct ir3_shader_variant *v, uint32_t gpu_id); +void * ir3_shader_assemble(struct ir3_shader_variant *v); struct ir3_shader_variant * ir3_shader_get_variant(struct ir3_shader *shader, struct ir3_shader_key *key, bool binning_pass, bool *created); struct ir3_shader * ir3_shader_from_nir(struct ir3_compiler *compiler, nir_shader *nir, diff --git a/src/gallium/drivers/freedreno/ir3/ir3_cmdline.c b/src/gallium/drivers/freedreno/ir3/ir3_cmdline.c index 7edee3086bf..a16c06872c3 100644 --- a/src/gallium/drivers/freedreno/ir3/ir3_cmdline.c +++ b/src/gallium/drivers/freedreno/ir3/ir3_cmdline.c @@ -58,7 +58,7 @@ static void dump_info(struct ir3_shader_variant *so, const char *str) { uint32_t *bin; const char *type = ir3_shader_stage(so); - bin = ir3_shader_assemble(so, so->shader->compiler->gpu_id); + bin = ir3_shader_assemble(so); debug_printf("; %s: %s\n", type, str); ir3_shader_disasm(so, bin, stdout); free(bin); -- 2.30.2