IMHO, the is_const function for Expr should call recursively based on the syntax of Expr.
@tqchen @ziheng @ajtulloch