On Thu, Nov 24, 2016 at 1:38 AM, Christopher Li <sparse@xxxxxxxxxxx> wrote: > In your patch: > > static struct token *builtin_types_compatible_p_expr(struct token *token, > struct expression **tree) > { > struct expression *expr = alloc_expression( > token->pos, EXPR_COMPARE); > - expr->flags = Int_const_expr; > expr->op = SPECIAL_EQUAL; > token = token->next; > if (!match_op(token, '(')) > > I think __builtin_xxx function expand similar to the processor > macro. In that sense, we might want to the consider this expression > as integer constant express as if the __builtin_xxx call is either 0 or 1. Never mind, I think you move it to the evaluation phase. 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