On Wed, Sep 2, 2009 at 3:35 PM, Kamil Dudka<kdudka@xxxxxxxxxx> wrote: > On Wednesday 02 of September 2009 21:19:49 Kamil Dudka wrote: > > The second holds. It's regression! I have some questions regarding your patch: Can we just set the expression->ctype to the enum type instead of adding the *enum_type? I think the current expr->ctype can be reached from enum_type->ctype.base_type any way. In other words, we do care about expression is enum type vs int type in this patch. After the type evaluation(and possible warning), we can convert that enum type back to the base int type because the back end does not care about enum. I think fixing the regression should be a separate patch from the this patch which adding new feature. It is easier to review as well. Sorry the rest of the patch will take me more time to go over. 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
