mirror of
https://gitlab.freedesktop.org/mesa/mesa.git
synced 2026-05-08 15:38:09 +02:00
ir_to_mesa: Allocate temporary instructions on the visitor's ralloc context
And don't delete them. Let ralloc clean them up. Deleting the
temporary IR leaves dangling references in the prog_instruction. That
results in a bad dereference when printing the IR with MESA_GLSL=dump.
NOTE: This is a candidate for the 7.10 and 7.11 branches.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=38584
Reviewed-by: Eric Anholt <eric@anholt.net>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
(cherry picked from commit dbda466fc0)
This commit is contained in:
parent
cb6dd6c399
commit
7424f9c1fe
1 changed files with 12 additions and 16 deletions
|
|
@ -827,48 +827,44 @@ ir_to_mesa_visitor::visit(ir_loop *ir)
|
|||
ir_dereference_variable *counter = NULL;
|
||||
|
||||
if (ir->counter != NULL)
|
||||
counter = new(ir) ir_dereference_variable(ir->counter);
|
||||
counter = new(mem_ctx) ir_dereference_variable(ir->counter);
|
||||
|
||||
if (ir->from != NULL) {
|
||||
assert(ir->counter != NULL);
|
||||
|
||||
ir_assignment *a = new(ir) ir_assignment(counter, ir->from, NULL);
|
||||
ir_assignment *a =
|
||||
new(mem_ctx) ir_assignment(counter, ir->from, NULL);
|
||||
|
||||
a->accept(this);
|
||||
delete a;
|
||||
}
|
||||
|
||||
ir_to_mesa_emit_op0(NULL, OPCODE_BGNLOOP);
|
||||
|
||||
if (ir->to) {
|
||||
ir_expression *e =
|
||||
new(ir) ir_expression(ir->cmp, glsl_type::bool_type,
|
||||
counter, ir->to);
|
||||
ir_if *if_stmt = new(ir) ir_if(e);
|
||||
new(mem_ctx) ir_expression(ir->cmp, glsl_type::bool_type,
|
||||
counter, ir->to);
|
||||
ir_if *if_stmt = new(mem_ctx) ir_if(e);
|
||||
|
||||
ir_loop_jump *brk = new(ir) ir_loop_jump(ir_loop_jump::jump_break);
|
||||
ir_loop_jump *brk =
|
||||
new(mem_ctx) ir_loop_jump(ir_loop_jump::jump_break);
|
||||
|
||||
if_stmt->then_instructions.push_tail(brk);
|
||||
|
||||
if_stmt->accept(this);
|
||||
|
||||
delete if_stmt;
|
||||
delete e;
|
||||
delete brk;
|
||||
}
|
||||
|
||||
visit_exec_list(&ir->body_instructions, this);
|
||||
|
||||
if (ir->increment) {
|
||||
ir_expression *e =
|
||||
new(ir) ir_expression(ir_binop_add, counter->type,
|
||||
counter, ir->increment);
|
||||
new(mem_ctx) ir_expression(ir_binop_add, counter->type,
|
||||
counter, ir->increment);
|
||||
|
||||
ir_assignment *a = new(ir) ir_assignment(counter, e, NULL);
|
||||
ir_assignment *a =
|
||||
new(mem_ctx) ir_assignment(counter, e, NULL);
|
||||
|
||||
a->accept(this);
|
||||
delete a;
|
||||
delete e;
|
||||
}
|
||||
|
||||
ir_to_mesa_emit_op0(NULL, OPCODE_ENDLOOP);
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue