Why not move setting TREE_SIDE_EFFECTS into build_target_expr, too? The one case where it isn't explicitly set currently will always have it set by build. Jason