diff options
Diffstat (limited to 'gcc/c-semantics.c')
-rw-r--r-- | gcc/c-semantics.c | 19 |
1 files changed, 6 insertions, 13 deletions
diff --git a/gcc/c-semantics.c b/gcc/c-semantics.c index 567a440..fe81249 100644 --- a/gcc/c-semantics.c +++ b/gcc/c-semantics.c @@ -149,19 +149,12 @@ build_stmt (enum tree_code code, ...) length = TREE_CODE_LENGTH (code); SET_EXPR_LOCATION (ret, input_location); - /* Most statements have implicit side effects all on their own, - such as control transfer. For those that do, we'll compute - the real value of TREE_SIDE_EFFECTS from its arguments. */ - switch (code) - { - case EXPR_STMT: - side_effects = false; - break; - default: - side_effects = true; - break; - } + /* TREE_SIDE_EFFECTS will already be set for statements with + implicit side effects. Here we make sure it is set for other + expressions by checking whether the parameters have side + effects. */ + side_effects = false; for (i = 0; i < length; i++) { tree t = va_arg (p, tree); @@ -170,7 +163,7 @@ build_stmt (enum tree_code code, ...) TREE_OPERAND (ret, i) = t; } - TREE_SIDE_EFFECTS (ret) = side_effects; + TREE_SIDE_EFFECTS (ret) |= side_effects; va_end (p); return ret; |