The code used to be like: pseudo_t phi = alloc_phi(...); phi->ident = phi->ident ? : ... but this new allocated phi can never have an identifier. Change this by removing the test and directly assigning the other part of the conditional. Signed-off-by: Luc Van Oostenryck <luc.vanoostenryck@xxxxxxxxx> --- flow.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/flow.c b/flow.c index 25a5bffbe..aa7a6586f 100644 --- a/flow.c +++ b/flow.c @@ -412,7 +412,7 @@ void add_dominator(struct pseudo_list **phi_list, struct instruction *insn, struct basic_block *bb = dom->bb; struct instruction *br = delete_last_instruction(&bb->insns); pseudo_t phi = alloc_phi(bb, dom->target, dom->type); - phi->ident = phi->ident ? : ident ? : dom->target->ident; + phi->ident = ident ? : dom->target->ident; add_instruction(&bb->insns, br); use_pseudo(insn, phi, add_pseudo(phi_list, phi)); } -- 2.12.0 -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
