From a6ad30202cacc933290f478d8f518a7b066268b6 Mon Sep 17 00:00:00 2001 From: Rob Clark Date: Tue, 3 May 2016 11:47:47 -0400 Subject: [PATCH] freedreno/ir3: remove a couple redundant is_flow()s Now that the opc's encode the instruction category (making them unique) we no longer need to check the category in addition to the opc. Signed-off-by: Rob Clark --- src/gallium/drivers/freedreno/ir3/ir3_depth.c | 2 +- src/gallium/drivers/freedreno/ir3/ir3_legalize.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/freedreno/ir3/ir3_depth.c b/src/gallium/drivers/freedreno/ir3/ir3_depth.c index c3f6de9..1b8a446 100644 --- a/src/gallium/drivers/freedreno/ir3/ir3_depth.c +++ b/src/gallium/drivers/freedreno/ir3/ir3_depth.c @@ -137,7 +137,7 @@ remove_unused_by_block(struct ir3_block *block) { list_for_each_entry_safe (struct ir3_instruction, instr, &block->instr_list, node) { if (!ir3_instr_check_mark(instr)) { - if (is_flow(instr) && (instr->opc == OPC_END)) + if (instr->opc == OPC_END) continue; /* mark it, in case it is input, so we can * remove unused inputs: diff --git a/src/gallium/drivers/freedreno/ir3/ir3_legalize.c b/src/gallium/drivers/freedreno/ir3/ir3_legalize.c index 7a49f4c..6acea01 100644 --- a/src/gallium/drivers/freedreno/ir3/ir3_legalize.c +++ b/src/gallium/drivers/freedreno/ir3/ir3_legalize.c @@ -292,7 +292,7 @@ resolve_dest_block(struct ir3_block *block) } else if (list_length(&block->instr_list) == 1) { struct ir3_instruction *instr = list_first_entry( &block->instr_list, struct ir3_instruction, node); - if (is_flow(instr) && (instr->opc == OPC_JUMP)) + if (instr->opc == OPC_JUMP) return block->successors[0]; } } -- 2.7.4