On Fri, Oct 29, 2010 at 6:22 AM, Bernd Petrovitsch <bernd@xxxxxxxxxx> wrote: > > Voila. - struct basic_block *true = br->bb_true; - struct basic_block *false = br->bb_false; - br->bb_false = true; - br->bb_true = false; + struct basic_block *true_expr = br->bb_true; + struct basic_block *false_expr = br->bb_false; + br->bb_false = true_expr; + br->bb_true = false_expr; Hmm, not very "true" symbol is an expression type. You end up calling basicl_block pointer as true_expr, that is misleading. Same thing happen to pseudo_t pointers. Looks fine otherwise. Chris -- 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
