fixes #733

Rather than assigning each node visitor manually, generic_visit checks if the node is any Expr and tries to optimize it, so Optimizer doesn't have to be kept in sync with CodeGenerator. There were some other nodes (Pair and Keyword) that had as_const but broke other things if they were folded, so this is limited to Expr, which seems to be the original intent anyway.