diff options
| author | Ian Lance Taylor <ian@airs.com> | 2005-04-25 01:18:23 +0000 |
|---|---|---|
| committer | Ian Lance Taylor <ian@gcc.gnu.org> | 2005-04-25 01:18:23 +0000 |
| commit | bbcfd969e6b35f9c4cad296f6c5e68f33a2ae0ba (patch) | |
| tree | b4b203ba10ff34644a7945213fdda6f4dbe0f18e /gcc/c-semantics.c | |
| parent | 9545759748856f86033141c5b341d8c1e7ecaeca (diff) | |
| download | gcc-bbcfd969e6b35f9c4cad296f6c5e68f33a2ae0ba.zip gcc-bbcfd969e6b35f9c4cad296f6c5e68f33a2ae0ba.tar.gz gcc-bbcfd969e6b35f9c4cad296f6c5e68f33a2ae0ba.tar.bz2 | |
c-semantics.c (build_stmt): Remove test of specific tree codes for TREE_SIDE_EFFECTS handling.
* c-semantics.c (build_stmt): Remove test of specific tree codes
for TREE_SIDE_EFFECTS handling.
From-SVN: r98693
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; |